[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