<div>Looking at it in the debugger, validateOutputConstraint and validateInputConstraint are also called earlier from Sema::ActOnAsmStmt after parsing.  It seems you are implying that the string literals should be modified at some point before the validate and other functions are called that use the strings.</div>

<div> </div>
<div>If so, can you tell me how to do it?  It seems that the expressions and so forth are meant to be immutable after being created.</div>
<div> </div>
<div>Otherwise, can you hold my hand a bit?  As I'm not sure what to do.</div>
<div> </div>
<div>-John</div>
<div><br> </div>
<div class="gmail_quote">On Fri, Jul 9, 2010 at 11:43 AM, 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">On Fri, Jul 9, 2010 at 11:27 AM, John Thompson<br>
<div class="im"><<a href="mailto:john.thompson.jtsoftware@gmail.com">john.thompson.jtsoftware@gmail.com</a>> wrote:<br></div>
<div class="im">>> 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>> Do you mean something like the enclosed patch?<br>
><br>> If so, may I check it in?<br><br></div>No, the check in question needs to happen in<br>CodeGenFunction::EmitAsmStmt, before we call validateOutputConstraint<br>or validateInputConstraint, or you won't end up with a valid asm in<br>
the IR.<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>