[PATCH] D87451: add new clang option -mno-xcoff-visibility

Digger via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 23 06:09:31 PDT 2020


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


================
Comment at: llvm/include/llvm/Target/TargetMachine.h:265
+  /// corresponding to -mno-xcoff-visibility.
+  bool getNoXCOFFVisibility() const { return Options.NoXCOFFVisibility; }
+
----------------
daltenty wrote:
> This seems like it needs the corresponding comand-line option for llc added and an llc test.
I think it will be in another separate  patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87451



More information about the llvm-commits mailing list