This is an archive of the discontinued LLVM Phabricator instance.

[BOLT] Upgrade to C++17
ClosedPublic

Authored by kazu on Aug 6 2022, 10:01 PM.

Details

Summary

Without this patch, I am getting errors like:

llvm-project/llvm/include/llvm/ADT/StringRef.h:233:7: error: use of
the 'nodiscard' attribute is a C++17 extension
[-Werror,-Wc++17-extensions]

Diff Detail

Event Timeline

kazu created this revision.Aug 6 2022, 10:01 PM
Herald added a reviewer: Amir. · View Herald Transcript
Herald added a reviewer: maksfb. · View Herald Transcript
Herald added a project: Restricted Project. · View Herald Transcript
kazu requested review of this revision.Aug 6 2022, 10:01 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 6 2022, 10:01 PM
Amir accepted this revision.Aug 7 2022, 7:18 PM
This revision is now accepted and ready to land.Aug 7 2022, 7:18 PM
This revision was automatically updated to reflect the committed changes.