[PATCH] D85474: Add -fbinutils-version= to gate ELF features on the specified binutils version

Rainer Orth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 13 10:41:45 PDT 2020


ro added inline comments.


================
Comment at: clang/lib/Driver/ToolChains/Clang.cpp:4642
+    int Num;
+    if (V == "future")
+      A->render(Args, CmdArgs);
----------------
MaskRay wrote:
> phosek wrote:
> > Another option would be `unstable`.
> I picked `future` because there is an precedent: `-mcpu=future` is used by some ppc folks.
I fear that's a terrible precedent: this name had to be chosen because for some unknown, but certainly silly, reason, IBM didn't wan't to call it `power10` before release.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85474



More information about the cfe-commits mailing list