[PATCH] D107304: [clangd][query-driver] Extract GCC version from the driver output

Aleksandr Platonov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 4 10:34:06 PDT 2021


ArcsinX added a comment.

In D107304#2924886 <https://reviews.llvm.org/D107304#2924886>, @kadircet wrote:

> Ok if you think that setting GCC version to higher values by default won't break anything,

I think that GCC version should be the same, which was used to build the project: it can be higher or lower than 4.2.1

In D107304#2925220 <https://reviews.llvm.org/D107304#2925220>, @joerg wrote:

> This discussion is becoming pointless.

Yes, seems we can't get agreement here, so I am planning to abandon this.

> Clang is not a 1:1 replacement for later GCC versions. Easiest example is the support for `__builtin_apply` and friends.

Still we have no example when GCC x.y.z can compile something, but clang with -fgnuc-version=x.y.z can't process this project with some critical errors (but works ok with -fgnuc-version=4.2.1). But it's easy to create an example when -fgnuc-version=4.2.1 doesn't work (example in the description).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107304



More information about the cfe-commits mailing list