ychen added inline comments.

================
Comment at: clang/lib/Driver/ToolChains/Arch/AArch64.cpp:223-224
                                        const llvm::Triple &Triple,
                                        const ArgList &Args,
+                                       llvm::opt::ArgStringList &CmdArgs,
                                        std::vector<StringRef> &Features,
----------------
Sorry if it has been discussed, why the compiler option is not added during 
constructing the clang job but here? All kinds of clients could call this 
method, only clang care about this option.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116221

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

Reply via email to