phosek added inline comments.

================
Comment at: clang/include/clang/Basic/LangOptions.def:353
 COMPATIBLE_VALUE_LANGOPT(MaxTokens, 32, 0, "Max number of tokens per TU or 0")
+LANGOPT(FuchsiaRelativeCXXABIVTables, 1, 0,
+        "Whether to use clang's relative C++ ABI "
----------------
I think we should omit the `Fuchsia` bit and make it a generic, this feature 
should be usable by any target.


================
Comment at: clang/include/clang/Driver/Options.td:1328
 
+def fexperimental_fuchsia_relative_cxx_abi_vtables : Flag<["-"], 
"fexperimental-fuchsia-relative-c++-abi-vtables">,
+  Group<f_Group>, Flags<[CC1Option]>,
----------------
Ditto, I'd drop the `fuchsia` bit from these flags.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72959



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to