[PATCH] D23650: Use target name instead of namespace as prefix for MCRegisterClasses array

Daniel Cederman via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 18 02:22:42 PDT 2016


dcederman created this revision.
dcederman added a subscriber: llvm-commits.
Herald added a subscriber: jyknight.

For Sparc the namespace (SP) is different from the target name (Sparc),
which causes the name of the array in this declaration to differ from
the name used in the definition.

https://reviews.llvm.org/D23650

Files:
  utils/TableGen/RegisterInfoEmitter.cpp

Index: utils/TableGen/RegisterInfoEmitter.cpp
===================================================================
--- utils/TableGen/RegisterInfoEmitter.cpp
+++ utils/TableGen/RegisterInfoEmitter.cpp
@@ -103,7 +103,7 @@
   OS << "namespace llvm {\n\n";
 
   OS << "class MCRegisterClass;\n"
-     << "extern const MCRegisterClass " << Namespace
+     << "extern const MCRegisterClass " << Target.getName()
      << "MCRegisterClasses[];\n\n";
 
   if (!Namespace.empty())


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D23650.68498.patch
Type: text/x-patch
Size: 472 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160818/3bbd80b3/attachment.bin>


More information about the llvm-commits mailing list