Page MenuHomePhabricator

[Driver] Update set of -std values with C++20 modules enabled
AcceptedPublic

Authored by ilya-biryukov on May 18 2022, 3:12 AM.

Details

Reviewers
sammccall
Summary

Previously missing are -std=gnu++2(0|a) and -std=(gnu|c)++2b.
This used to result in wrong -cc1 flags produced by Driver.

Diff Detail

Event Timeline

ilya-biryukov created this revision.May 18 2022, 3:12 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 18 2022, 3:12 AM
ilya-biryukov requested review of this revision.May 18 2022, 3:12 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 18 2022, 3:12 AM
Herald added a subscriber: MaskRay. · View Herald Transcript
sammccall accepted this revision.May 18 2022, 5:17 AM

LG, but can you change the patch description as you're bringing the set up to date rather than just adding gnu++20

This revision is now accepted and ready to land.May 18 2022, 5:17 AM
ilya-biryukov retitled this revision from [Driver] Recognize -std=gnu++20 enables C++ modules to [Driver] Update set of -std values with C++20 modules enabled.May 18 2022, 5:23 AM
ilya-biryukov edited the summary of this revision. (Show Details)
ilya-biryukov edited the summary of this revision. (Show Details)