[PATCH] D88737: [AIX] Turn -fdata-sections on by default in Clang

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Oct 3 15:14:19 PDT 2020


MaskRay added inline comments.


================
Comment at: lld/Common/TargetOptionsCommandFlags.cpp:17
+llvm::TargetOptions
+lld::initTargetOptionsFromCodeGenFlags(const llvm::Triple &TheTriple) {
+  return llvm::codegen::InitTargetOptionsFromCodeGenFlags(TheTriple);
----------------
Currently lld does not need Triple. Consider not changing the signature of `initTargetOptionsFromCodeGenFlags`.


================
Comment at: llvm/include/llvm/CodeGen/CommandFlags.h:142
+///    passed in.
+TargetOptions InitTargetOptionsFromCodeGenFlags(const llvm::Triple &TheTriple);
 
----------------
If you are going to change the signature, consider renaming it to InitTargetOptionsFromCodeGenFlags to be consistent with the coding standard.


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

https://reviews.llvm.org/D88737



More information about the cfe-commits mailing list