[PATCH] D133028: [amdgpu] Always, instead of mostly, remove unused LDS symbols

Joseph Huber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 31 09:19:38 PDT 2022


jhuber6 added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPULowerModuleLDSPass.cpp:248-254
+    for (auto &GV : make_early_inc_range(M.globals())) {
+      if (AMDGPU::isLDSVariableToLower(GV)) {
+        if (GV.use_empty()) {
+          GV.eraseFromParent();
+        }
+      }
+    }
----------------
Nit. formatting.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133028



More information about the llvm-commits mailing list