[llvm] [AMDGPU] Localize resource info symbols (PR #185732)

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 11 05:33:03 PDT 2026


================
@@ -18,17 +18,23 @@
 #include "llvm/MC/MCAsmInfo.h"
 #include "llvm/MC/MCContext.h"
 #include "llvm/MC/MCSymbol.h"
+#include "llvm/Support/CommandLine.h"
 #include "llvm/Target/TargetMachine.h"
 
 #define DEBUG_TYPE "amdgpu-mc-resource-usage"
 
 using namespace llvm;
 
+static cl::opt<bool> LocalizeResourceSymbols(
----------------
arsenm wrote:

cl::opts are not for user facing controls. Adding this will only introduce trouble moving towards supporting object linking.

The symbol has a visibility and all associated metadata should have the same visibility 

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


More information about the llvm-commits mailing list