<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><div>On Aug 10, 2010, at 5:43 PM, John Thompson wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div>Apparently, "=r,m" constraints are legal. This fixes pr7869. Permission to check-in?</div></blockquote><div><br></div>Ok with a testcase and no regressions on the testsuite.</div><div><br></div><div>-Chris</div><div><br><blockquote type="cite">
<div> </div>
<div>Index: lib/Basic/TargetInfo.cpp<br>===================================================================<br>--- lib/Basic/TargetInfo.cpp (revision 110706)<br>+++ lib/Basic/TargetInfo.cpp (working copy)<br>@@ -289,11 +289,9 @@<br>
break;<br> case ',': // multiple alternative constraint. Pass it.<br> Name++;<br>- // An output constraint must start with '=' or '+'<br>- if (*Name != '=' && *Name != '+')<br>
- return false;<br>- if (*Name == '+')<br>- Info.setIsReadWrite();<br>+ // Handle additional optional '=' or '+' modifiers.<br>+ if (*Name == '=' || *Name == '+')<br>
+ Name++;<br> break;<br> case '?': // Disparage slightly code.<br> case '!': // Disparage severly.<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></div>
_______________________________________________<br>cfe-commits mailing list<br><a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits<br></blockquote></div><br></body></html>