This is an archive of the discontinued LLVM Phabricator instance.

[flang] Fixed out-of-tree build after D156435.
ClosedPublic

Authored by vzakhari on Aug 1 2023, 9:54 AM.

Details

Summary

I decided to clean up the CMake files as well.
Only FotranEvaluate directly needs quadmath, so we only need
to link these two together.

Diff Detail

Event Timeline

vzakhari created this revision.Aug 1 2023, 9:54 AM
Herald added a project: Restricted Project. · View Herald Transcript
vzakhari requested review of this revision.Aug 1 2023, 9:54 AM
PeteSteinfeld accepted this revision.Aug 1 2023, 10:08 AM

Thanks for cleaning this up, Slava.

All builds and tests correctly and looks good.

This revision is now accepted and ready to land.Aug 1 2023, 10:08 AM
Renaud-K added inline comments.Aug 1 2023, 10:09 AM
flang/lib/Evaluate/CMakeLists.txt
25

Can we request a static library only.

vzakhari added inline comments.Aug 1 2023, 10:19 AM
flang/lib/Evaluate/CMakeLists.txt
25

Sounds reasonable to me. I will make it in a separate commit. Thank you!

This revision was landed with ongoing or failed builds.Aug 1 2023, 10:52 AM
This revision was automatically updated to reflect the committed changes.