Skip to content

Commit be7e411

Browse files
authored
[NFC] Add comment to describe the intention use of newly added avail-extern-gv-in-addrspace-to-local (#144911)
1 parent ad36800 commit be7e411

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

llvm/lib/Transforms/IPO/ElimAvailExtern.cpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,10 @@ static cl::opt<bool> ConvertToLocal(
3535
cl::desc("Convert available_externally into locals, renaming them "
3636
"to avoid link-time clashes."));
3737

38+
// This option was originally introduced to correctly support the lowering of
39+
// LDS variables for AMDGPU when ThinLTO is enabled. It can be utilized for
40+
// other purposes, but make sure it is safe to do so, as privatizing global
41+
// variables is generally not safe.
3842
static cl::opt<unsigned> ConvertGlobalVariableInAddrSpace(
3943
"avail-extern-gv-in-addrspace-to-local", cl::Hidden,
4044
cl::desc(

0 commit comments

Comments
 (0)