[PATCH] D76932: [AIX] emit .extern and .weak directive linkage

[Jason Liu via Phabricator via cfe-commits]

jasonliu added inline comments.


================
Comment at: llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp:1498
+      }
+      emitLinkage(&F, Name);
+      continue;
----------------
jasonliu wrote:
> I'm surprised we do not enter here for `foo_ext_weak()`. The result of that is we need to do something in `lowerConstant()`, which I would want to avoid if possible. 
> Should we look into why `isDeclarationForLinker()` returns false for `foo_ext_weak()` here?
> My guess is we returned false for `!F->isMaterializable()`. And I'm not sure if `foo_ext_weak` is materializable is the correct answer here. 
Sorry, ignore the above comment. I think we did entered here for emitLinkage for `foo_ext_weak`. But the hasContainingCsect query blocked the emission there. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76932