[llvm] [NVPTX] improve identifier renaming for PTX (PR #79459)

Alex MacLean via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 25 11:48:24 PST 2024


================
@@ -73,10 +74,10 @@ std::string NVPTXAssignValidGlobalNames::cleanUpName(StringRef Name) {
   std::string ValidName;
   raw_string_ostream ValidNameStream(ValidName);
   for (char C : Name) {
-    if (C == '.' || C == '@' || C == '<' || C == '>') {
-      ValidNameStream << "_$_";
-    } else {
+    if (isAlnum(C) || C == '_' || C == '$') {
       ValidNameStream << C;
+    } else {
+      ValidNameStream << "_$_";
----------------
AlexMaclean wrote:

This renaming is already limited to names with local linkage.  

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


More information about the llvm-commits mailing list