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