[PATCH] D91913: Suppress non-conforming GNU paste extension in all standard-conforming modes

Nico Weber via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 25 13:32:14 PST 2021


thakis added a comment.

In D91913#2520503 <https://reviews.llvm.org/D91913#2520503>, @hvdijk wrote:

> In D91913#2520432 <https://reviews.llvm.org/D91913#2520432>, @zequanwu wrote:
>
>> In D91913#2520414 <https://reviews.llvm.org/D91913#2520414>, @hvdijk wrote:
>>
>>> In D91913#2520399 <https://reviews.llvm.org/D91913#2520399>, @zequanwu wrote:
>>>
>>>> This change also breaks many chrome ToT bots(not just windows bot), example build: https://ci.chromium.org/ui/p/chrome/builders/ci/ToTLinuxOfficial/10524/steps?succeeded=true&debug=false
>>>
>>> That requires an account to see.
>>
>> This should be accessible without an account: https://logs.chromium.org/logs/chromium/buildbucket/cr-buildbucket.appspot.com/8857139118288337920/+/steps/compile/0/stdout
>
> Indeed. However, here, this seems to be a user problem. Chrome `-std=c++*` when building with clang, but `-std=gnu++*` when building with GCC for exactly this reason, see build/config/compiler/BUILD.gn. Chrome should use `-std=gnu++*` with clang too.

We (and every other project out there) needs some incremental rollout plan for this. We don't want to allow _more_ GNU features. Please put this behind a `-f` toggle, or revert this for a bit implementing this takes some time.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D91913/new/

https://reviews.llvm.org/D91913



More information about the cfe-commits mailing list