<div>Oh right, I didn't even notice the double character constants.</div>
<div> </div>
<div>Here's another whack it.  Since the function is passed a reference to the string pointer, it seems I can bump it.  It this an acceptable side effect?</div>
<div> </div>
<div>Thanks for looking it.</div>
<div> </div>
<div>-John<br><br></div>
<div class="gmail_quote">On Tue, Jun 22, 2010 at 10:23 PM, Chris Lattner <span dir="ltr"><<a href="mailto:clattner@apple.com">clattner@apple.com</a>></span> wrote:<br>
<blockquote style="BORDER-LEFT: #ccc 1px solid; MARGIN: 0px 0px 0px 0.8ex; PADDING-LEFT: 1ex" class="gmail_quote">
<div style="WORD-WRAP: break-word">
<div>
<div></div>
<div class="h5"><br>
<div>
<div>On Jun 22, 2010, at 2:41 PM, John Thompson wrote:</div><br>
<blockquote type="cite">
<div>The enclosed patch contains three changes:</div>
<div> </div>
<div>1.  In compiling the afore-mentioned code targeting PS3, I ran into the lack of some needed implicit defines.<br clear="all"></div>
<div>2. Changes to the PS3 target class to fix some sizes of things.  Curiously, the PS3 thinks it's a PowerPC64, though it's configuration seems to actually be more like PowerPC32.</div>
<div> </div>
<div>3. Added some PPC-specific constraints, based on the gcc docs.  I just guessed at what the actions should be.  This is to fix some code I'm trying to compiler which uses at least the Z constraint.</div>
<div> </div>
<div>With your permission I will check it in.  Previously I was given permission to mess with the PS3 stuff, but because the constraint thing affects the PowerPC, I thought I should ask.</div>
<div> </div></blockquote><br></div></div></div>
<div>This looks ok to me, except for:</div>
<div><br></div>
<div>
<div style="MARGIN: 0px">+    case 'wd':// VSX vector register to hold vector double data </div>
<div style="MARGIN: 0px">+    case 'wf':// VSX vector register to hold vector float data </div>
<div style="MARGIN: 0px">+    case 'ws':// VSX vector register to hold scalar float data </div>
<div style="MARGIN: 0px">+    case 'wa':// Any VSX register </div>
<div><font face="Inconsolata"><br></font></div>
<div><font face="Inconsolata"><br></font></div>
<div><font face="Inconsolata">That's probably not doing what you think it's doing...</font></div>
<div><font face="Inconsolata"><br></font></div>
<div><font face="Inconsolata">-Chris</font></div></div><br></div></blockquote></div><br><br clear="all"><br>-- <br>John Thompson<br><a href="mailto:John.Thompson.JTSoftware@gmail.com">John.Thompson.JTSoftware@gmail.com</a><br>
<br>