This is an archive of the discontinued LLVM Phabricator instance.

[compiler-rt] Always use C++17 when building compiler-rt standalone
AbandonedPublic

Authored by glandium on Aug 15 2022, 7:52 PM.

Details

Reviewers
None
Summary

Platforms where -std=c++17 is not a valid flag (e.g. Windows with
clang-cl) are currently not getting the flag. This makes the setup for
compiler-rt similar to other subprojects.

Diff Detail

Event Timeline

glandium created this revision.Aug 15 2022, 7:52 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 15 2022, 7:52 PM
glandium requested review of this revision.Aug 15 2022, 7:52 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 15 2022, 7:52 PM
Herald added a subscriber: Restricted Project. · View Herald Transcript
glandium retitled this revision from Always use C++17 when building compiler-rt standalone to [compiler-rt] always use C++17 when building compiler-rt standalone.Aug 15 2022, 7:55 PM
glandium retitled this revision from [compiler-rt] always use C++17 when building compiler-rt standalone to [compiler-rt] Always use C++17 when building compiler-rt standalone.

@smeenai I see you pushed something similar in 5737f6a527d782e6577e5cc1e0c743df2c98546a, but missing the config-ix.cmake part, would you mind pushing that cleanup too?

Ah, sorry, I didn't realize you already had this change up. I pushed rGd7a4ff9986c8dcaf967770c868e9020d3f328e9f.

glandium abandoned this revision.Aug 18 2022, 4:36 PM

Thank you