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

Harald van Dijk via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 26 02:12:00 PST 2021


hvdijk added a comment.

Restoring the old behaviour for LLVM 12 may or may not help the Chrome/Chromium people: if they also regularly build with clang from git, they would need to handle this change anyway. However, if it does help, then it sounds like a good idea to me.

> So we should be working towards removing the `getNumParams() < 2` check in at least the C++20 case.

Right, and the same change has also been accepted for C, has it not? If it has, then also the C2x case.

It would be good to align with GCC here. I would imagine that they will want the same thing for GCC you are proposing for clang, but it would be good to make sure. Specifically: it would be good to confirm that there are no objections from GCC's side to supporting `#ifdef __VA_OPT__` as well.


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