[clang] [llvm] [AMDGPU][LTO] Assume closed world after linking (PR #105845)

Shilei Tian via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 23 09:08:39 PDT 2024


================
@@ -761,7 +761,8 @@ void AMDGPUTargetMachine::registerPassBuilderCallbacks(PassBuilder &PB) {
         if (EnableLowerModuleLDS)
           PM.addPass(AMDGPULowerModuleLDSPass(*this));
         if (EnableAMDGPUAttributor && Level != OptimizationLevel::O0)
-          PM.addPass(AMDGPUAttributorPass(*this));
+          PM.addPass(
+              AMDGPUAttributorPass(*this, AMDGPUAttributorOptions{true}));
----------------
shiltian wrote:

I'd suggest to do:

```
AMDGPUAttributorOptions Options;
Options.IsClosedWorld = true;
```

And then pass the option.

https://github.com/llvm/llvm-project/pull/105845


More information about the llvm-commits mailing list