[PATCH] R600/SI: Fix verifier error with pseudo store instructions.

Matt Arsenault Matthew.Arsenault at amd.com
Wed Apr 30 14:52:16 PDT 2014


Use i32 instead of specifying SReg_32. When this is 
the pseudo INDIRECT_BASE_ADDR, this would give a bogus
verifier error.

No test since I for some reason haven't reproduced this on trunk without a set of patches that I've been working on.

http://reviews.llvm.org/D3577

Files:
  lib/Target/R600/SIInstrInfo.td

Index: lib/Target/R600/SIInstrInfo.td
===================================================================
--- lib/Target/R600/SIInstrInfo.td
+++ lib/Target/R600/SIInstrInfo.td
@@ -137,7 +137,7 @@
 }]>;
 
 def FRAMEri32 : Operand<iPTR> {
-  let MIOperandInfo = (ops SReg_32:$ptr, i32imm:$index);
+  let MIOperandInfo = (ops i32:$ptr, i32imm:$index);
 }
 
 //===----------------------------------------------------------------------===//
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D3577.8993.patch
Type: text/x-patch
Size: 436 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140430/6690436f/attachment.bin>


More information about the llvm-commits mailing list