<div>> As a first<br>approximation, we could just chop off everything after the first comma<br>(which would be enough to work for PR7338).<br></div>
<div>Do you mean something like the enclosed patch?</div>
<div> </div>
<div>If so, may I check it in?</div>
<div> </div>
<div>-John<br></div>
<div class="gmail_quote">On Thu, Jul 8, 2010 at 10:55 PM, Eli Friedman <span dir="ltr"><<a href="mailto:eli.friedman@gmail.com">eli.friedman@gmail.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 class="im">On Thu, Jul 8, 2010 at 10:02 PM, John Thompson<br><<a href="mailto:john.thompson.jtsoftware@gmail.com">john.thompson.jtsoftware@gmail.com</a>> wrote:<br>> Eli,<br>><br>> I've reverted it.<br>
><br>> But would you either please reconsider or give me an alternative solution?<br>><br>> A comment in bug 7338 mentions input from Andrew Pinsky, who I think is one<br>> of the gcc guys, that gcc does ignore the commas.  But perhaps he just meant<br>
> our PS3 gcc.  If so, perhaps I could do it only for the PS3 target, rather<br>> than for the PPC target, as I originally did?<br>> The code in question using the commas is in the PS3 SDK headers, meaning<br>> it's not so easy to change.<br>
<br></div>I don't think he meant gcc ignores commas in general, just that in<br>that particular case, the code is equivalent with and without commas.<br>The correct solution is not in platform specific code; it's something<br>
like <a href="http://llvm.org/bugs/show_bug.cgi?id=7338#c6" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=7338#c6</a>.  As a first<br>approximation, we could just chop off everything after the first comma<br>(which would be enough to work for PR7338).<br>
<font color="#888888"><br>-Eli<br></font></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>