On Fri, Jan 13, 2012 at 5:20 PM, Eli Friedman <[email protected]> wrote:
> On Fri, Jan 13, 2012 at 2:05 PM, Sebastian Pop <[email protected]> wrote:
>> Hi Eli,
>>
>> On Fri, Jan 13, 2012 at 3:38 PM, Eli Friedman <[email protected]> wrote:
>>> I had to revert this in r148141; it was causing regression test
>>> failures on Mac buildbots.  A transcript of the failures follows:
>>>
>>
>> Thanks for reporting these fails.  Are these the only fails on the
>> buildbot?
>
> Yes, just those two.
>
>> I would need some help on understanding what is a normal output and
>> what is erroneous in this log file: i.e., what went wrong.
>
> the noexecstack.c test is crashing.  I haven't looked closely at the other 
> test.

I see that -triple flag is parsed somewhere in the clang front-end, so
I removed the -triple alias that I added:
+def triple : Flag<"-triple">, Alias<target>;

Here is the wrong change in bindings.c:
-// RUN: %clang -ccc-host-triple i386-unknown-unknown
-ccc-print-bindings -ccc-no-clang-cpp -fsyntax-only -no-integrated-cpp
%s 2> %t
+// RUN:target -target i386-unknown-unknown -ccc-print-bindings
-ccc-no-clang-cpp -fsyntax-only -no-integrated-cpp %s 2> %t

It seems like I replaced %clang with target, so this test wouldn't pass.
I am committing the amended patch.

Thanks,
Sebastian
--
Qualcomm Innovation Center, Inc is a member of Code Aurora Forum

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to