leonardchan marked an inline comment as done.
leonardchan added inline comments.


================
Comment at: clang/lib/Frontend/CompilerInvocation.cpp:3469
+
+  // The value can be empty, which indicates the system default should be used.
+  StringRef CXXABI = Args.getLastArgValue(OPT_fcxx_abi_EQ);
----------------
phosek wrote:
> Other flags like `-funwindlib` use either empty string or `"platform"`, 
> perhaps we should support the same for `-fc++-abi`?
Would an empty string already imply the platform default ABI? If so, I believe 
this already handles that case in `ASTContext::getCXXABIKind` when getting the 
ABI kind.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85802

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

Reply via email to