<div>Thanks for catching this.  I'll add a check for tabs and line endings as part of my submission procedure.  I've enclosed an updated patch.</div>
<div> </div>
<div>May I check this in?</div>
<div> </div>
<div>-John<br><br></div>
<div class="gmail_quote">On Tue, Oct 12, 2010 at 1:32 AM, Frits van Bommel <span dir="ltr"><<a href="mailto:fvbommel@gmail.com">fvbommel@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 Tue, Oct 12, 2010 at 2:16 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">> Here's an updated patch with some more revisions, adding some missing stuff<br>> I found while developing new tests for the new stuff.  Still lacking is that<br>> I also see that the "lowering" operation is missing support for some of the<br>
> x86 constraints I added, which I'll work on next, as well as submitting the<br>> new tests for this stuff.  But may I check in the current changes, so as not<br>> to diverge too much from the trunk?  I've successfully run the regression<br>
> tests, test-suite, and the gcc tests with Clang on Linux 64-bit with the<br>> current changes.<br><br></div>You have hard tabs inside "enum ConstraintWeight" (in<br>TargetLowering.h, at the top of the patch).<br>
</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>