This is an archive of the discontinued LLVM Phabricator instance.

[libc] Split out FPExceptMatcher from the FP utils
ClosedPublic

Authored by jhuber6 on May 3 2023, 6:39 AM.

Details

Summary

The FPEceptMatcher.cpp file uses system utilities and includes C++
libraries. This patch pulls it out of the main FPTestHelpers target so
we can exclude it from hermetic only tests.

Depends on D149705

Diff Detail

Event Timeline

jhuber6 created this revision.May 3 2023, 6:39 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptMay 3 2023, 6:39 AM
jhuber6 requested review of this revision.May 3 2023, 6:39 AM
sivachandra accepted this revision.May 3 2023, 9:10 AM
sivachandra added inline comments.
libc/test/src/CMakeLists.txt
30 ↗(On Diff #519055)

Instead of adding LibcFPExceptionHelpers here for all math tests, WDYT of adding to LINK_LIBRARIES of the two tests which actually use this?

This revision is now accepted and ready to land.May 3 2023, 9:10 AM
jhuber6 added inline comments.May 3 2023, 9:10 AM
libc/test/src/CMakeLists.txt
30 ↗(On Diff #519055)

Works for me.

jhuber6 updated this revision to Diff 519110.May 3 2023, 9:19 AM

Change to explicit link libraries

This revision was automatically updated to reflect the committed changes.