[llvm] [WIP][RFC] Teach MCA constant registers do not create dependencies (PR #89387)

Rin Dobrescu via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 23 03:55:43 PDT 2024


================
@@ -962,7 +962,7 @@ void RegisterInfoEmitter::runMCDesc(raw_ostream &OS, CodeGenTarget &Target,
 
   OS << "extern const MCRegisterDesc " << TargetName
      << "RegDesc[] = { // Descriptors\n";
-  OS << "  { " << RegStrings.get("") << ", 0, 0, 0, 0, 0 },\n";
+  OS << "  { " << RegStrings.get("") << ", 0, 0, 0, 0, 0, 0 },\n";
----------------
Rin18 wrote:

I understand why it worked without this change, my question was more regarding whether or not this is even worth changing. Like you said, the bool IsConstant gets zero-initialised anyway. I only added this in to keep things looking consistent, which I wasn't sure was enough of a reason to commit this change.  

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


More information about the llvm-commits mailing list