<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><div>On Nov 16, 2012, at 12:38 AM, Joe Abbey <<a href="mailto:jabbey@arxan.com">jabbey@arxan.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><meta http-equiv="Content-Type" content="text/html charset=windows-1252"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div>Hal (et al),</div><div><br></div><div>One of my build bots reported a warning:</div><div><pre style="font-family: 'Courier New', courier, monotype, monospace; "><span class="stdout">PPCRegisterInfo.cpp:501:51: warning: cast from type ‘const llvm::MachineFunction*’ to type ‘llvm::MachineFunction*’ casts away qualifiers [-Wcast-qual]</span></pre></div><div>Index: lib/Target/PowerPC/PPCRegisterInfo.cpp</div><div>===================================================================</div><div>--- lib/Target/PowerPC/PPCRegisterInfo.cpp<span class="Apple-tab-span" style="white-space:pre"> </span>(revision 168110)</div><div>+++ lib/Target/PowerPC/PPCRegisterInfo.cpp<span class="Apple-tab-span" style="white-space:pre"> </span>(working copy)</div><div>@@ -498,7 +498,8 @@</div><div> } else if (CRSpillFrameIdx) {</div><div> FrameIdx = CRSpillFrameIdx;</div><div> } else {</div><div>- MachineFrameInfo *MFI = ((MachineFunction &)MF).getFrameInfo();</div><div>+ MachineFrameInfo *MFI = </div><div>+ (const_cast<MachineFunction &>MF).getFrameInfo();</div><div> FrameIdx = MFI->CreateFixedObject((uint64_t)4, (int64_t)-4, true);</div><div> CRSpillFrameIdx = FrameIdx;</div><div> }</div><div><br></div><div>Ok to commit?</div></div></blockquote><br></div><div>Past Joe,</div><div><br></div><div>Make sure your current build is setup to do PowerPC builds.</div><div><br></div><div><div>Also, swapping a warning for an error is a horrible trade… let's try that patch again:</div><div><br></div><div><div>Index: lib/Target/PowerPC/PPCRegisterInfo.cpp</div><div>===================================================================</div><div>--- lib/Target/PowerPC/PPCRegisterInfo.cpp<span class="Apple-tab-span" style="white-space: pre; "> </span>(revision 168110)</div><div>+++ lib/Target/PowerPC/PPCRegisterInfo.cpp<span class="Apple-tab-span" style="white-space: pre; "> </span>(working copy)</div><div>@@ -498,7 +498,8 @@</div><div> } else if (CRSpillFrameIdx) {</div><div> FrameIdx = CRSpillFrameIdx;</div><div> } else {</div><div>- MachineFrameInfo *MFI = ((MachineFunction &)MF).getFrameInfo();</div><div>+ MachineFrameInfo *MFI = </div><div>+ (const_cast<MachineFunction &>(MF)).getFrameInfo();</div><div> FrameIdx = MFI->CreateFixedObject((uint64_t)4, (int64_t)-4, true);</div><div> CRSpillFrameIdx = FrameIdx;</div><div> }</div></div></div><br><div>Joe</div></body></html>