[PATCH] D83875: [XCOFF][AIX] Enable -ffunction-sections

Jason Liu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 27 14:41:13 PDT 2020


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


================
Comment at: llvm/test/CodeGen/PowerPC/aix-xcoff-funcsect.ll:15
+
+define hidden void @hidden_foo() {
+entry:
----------------
Xiangling_L wrote:
> Just a question here, why would we want to test `hidden_foo` and `static_overalign_foo`? I didn't see anything special about them when handling `-ffunction-sections` options.
I want to make sure `emitLinkage` and `setCsectAlignment` in `doInitialization` still works with function-sections. As some of them were accepting labels before, and setCsectAlignment was only doing csect alignment for the whole `.text` section. 


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

https://reviews.llvm.org/D83875





More information about the llvm-commits mailing list