[PATCH] D107280: [RISCV] Lower extern_weak symbols using the GOT for the medany model

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:2759
+  bool IsExternWeak = GV->hasExternalWeakLinkage();
+  SDValue Addr = getAddr(N, DAG, IsLocal, IsExternWeak);
 
----------------
jrtc27 wrote:
> MaskRay wrote:
> > IsExternWeak can be omitted
> What do you mean? If it's omitted then it defaults to false and we get the old codegen
Change the argument. The variable can be removed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107280