[PATCH] D87451: add new option ignore-xcoff-visibility
Digger via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Oct 2 08:00:35 PDT 2020
DiggerLin marked 5 inline comments as done.
DiggerLin added inline comments.
================
Comment at: clang/test/CodeGen/aix-no-xcoff-visibility.cpp:75
+
+// VISIBILITY-IR: @b = protected global i32 0
+// VISIBILITY-IR: @pramb = hidden global i32 0
----------------
jasonliu wrote:
> Not sure if the IR check is really necessary, since we haven't made any IR change here. It's going to be all the same with or without the new -m option.
IR output will not change in the new option. I think we need to check whether the IR be changed in the new option
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D87451/new/
https://reviews.llvm.org/D87451
More information about the cfe-commits
mailing list