[PATCH] D74749: ThinLTOBitcodeWriter: drop dso_local when a GlobalVariable is converted to a declaration

[Fangrui Song via Phabricator via llvm-commits]

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


================
Comment at: llvm/lib/Transforms/IPO/FunctionImport.cpp:993
     GV.replaceAllUsesWith(NewGV);
     return false;
   }
----------------
tejohnson wrote:
> Don't we also need it here (alias case)?
GlobalAlias does not need it, but I'll add a test case that an `alias` works.

I will also add a hidden GlobalVariable case. If we don't check `if (!GV.isImplicitDSOLocal())`, setting a hidden GlobalVariable declaration dso_preemptable will trigger a verifier error (`IR/Verifier.cpp:visitGlobalValue`).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74749