[PATCH] D75866: [AIX] supporting the visibility attribute for aix assembly
Jason Liu via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed May 27 13:36:35 PDT 2020
jasonliu added inline comments.
================
Comment at: llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp:1521
emitLinkage(&F, Name);
+ continue;
}
----------------
jasonliu wrote:
> ```
> if (TM.getTargetTriple().isOSBinFormatXCOFF() && !F.isIntrinsic()) {
> ...
> emitLinkage()
> } else {
> ...
> emitVisibility()
> }
> ```
>
> Might be a better choice than "continue" here. If F is an intrinsic then it would go to "emitVisibility" block even if it's XCOFF target. It may or may not do any harm, but I think it would be better not go to there.
>
Sorry, I think this might be better.
```
if (TM.getTargetTriple().isOSBinFormatXCOFF())
if (!F.isIntrinsic())
continue;
...
emitLinkage()
} else {
...
emitVisibility()
}
```
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75866/new/
https://reviews.llvm.org/D75866
More information about the llvm-commits
mailing list