<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br><div><div>On Apr 30, 2014, at 3:00 PM, Tom Stellard <<a href="mailto:tom@stellard.net">tom@stellard.net</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;">On Wed, Apr 30, 2014 at 09:52:16PM +0000, Matt Arsenault wrote:<br><blockquote type="cite">Use i32 instead of specifying SReg_32. When this is<span class="Apple-converted-space"> </span><br>the pseudo INDIRECT_BASE_ADDR, this would give a bogus<br>verifier error.<br><br>No test since I for some reason haven't reproduced this on trunk without a set of patches that I've been working on.<br><br></blockquote><br>Does this allow you to enable the -verify-machineinstrs flag for these tests:<br><br>LLVM :: CodeGen/R600/array-ptr-calc-i32.ll<br>LLVM :: CodeGen/R600/insert_vector_elt.ll<br>LLVM :: CodeGen/R600/private-memory.ll<br><br>-Tom<br><br></div></blockquote><div><br></div><div>Yes, I will add it to those tests</div><div><br></div><div><br></div><br><blockquote type="cite"><div style="font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><br><blockquote type="cite"><a href="http://reviews.llvm.org/D3577">http://reviews.llvm.org/D3577</a><br><br>Files:<br> lib/Target/R600/SIInstrInfo.td<br><br>Index: lib/Target/R600/SIInstrInfo.td<br>===================================================================<br>--- lib/Target/R600/SIInstrInfo.td<br>+++ lib/Target/R600/SIInstrInfo.td<br>@@ -137,7 +137,7 @@<br>}]>;<br><br>def FRAMEri32 : Operand<iPTR> {<br>- let MIOperandInfo = (ops SReg_32:$ptr, i32imm:$index);<br>+ let MIOperandInfo = (ops i32:$ptr, i32imm:$index);<br>}<br><br>//===----------------------------------------------------------------------===//<br></blockquote><br><blockquote type="cite">Index: lib/Target/R600/SIInstrInfo.td<br>===================================================================<br>--- lib/Target/R600/SIInstrInfo.td<br>+++ lib/Target/R600/SIInstrInfo.td<br>@@ -137,7 +137,7 @@<br>}]>;<br><br>def FRAMEri32 : Operand<iPTR> {<br>- let MIOperandInfo = (ops SReg_32:$ptr, i32imm:$index);<br>+ let MIOperandInfo = (ops i32:$ptr, i32imm:$index);<br>}<br><br>//===----------------------------------------------------------------------===//<br></blockquote><br><blockquote type="cite">_______________________________________________<br>llvm-commits mailing list<br><a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br><a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br></blockquote><br>_______________________________________________<br>llvm-commits mailing list<br><a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br><a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a></div></blockquote></div><br></body></html>