[cfe-commits] r99197 - /cfe/trunk/test/Headers/c89.c

[Daniel Dunbar]

On Fri, Mar 26, 2010 at 11:29 AM, Chris Lattner <clattner at apple.com> wrote:
>
> On Mar 25, 2010, at 4:01 PM, Daniel Dunbar wrote:
>
>> Please send me a test case.
>
> Testcase here:
> http://google1.osuosl.org:8011/builders/clang-i686-xp-msvc9/builds/5109

-triple isn't a clang option, which, as it turns out, is exactly what
the log tells you.

 - Daniel

> :)
>
> -Chris
>
>>
>> - Daniel
>>
>> On Thu, Mar 25, 2010 at 9:39 AM, Chris Lattner <clattner at apple.com> wrote:
>>>
>>> On Mar 25, 2010, at 8:51 AM, Daniel Dunbar wrote:
>>>
>>>> On Mon, Mar 22, 2010 at 12:46 PM, Chris Lattner <clattner at apple.com> wrote:
>>>>>
>>>>> On Mar 22, 2010, at 11:51 AM, Chris Lattner wrote:
>>>>>
>>>>>> Author: lattner
>>>>>> Date: Mon Mar 22 13:51:09 2010
>>>>>> New Revision: 99197
>>>>>>
>>>>>> URL: http://llvm.org/viewvc/llvm-project?rev=99197&view=rev
>>>>>> Log:
>>>>>> hopefully fix this on non-sse2 hosts.  Shouldn't an apple-darwin10
>>>>>> triple imply sse2?
>>>>>
>>>>> Daniel, do shouldn't a "darwin" triple imply SSE2, even on the MSVC builder?  Is this a bug or am I missing something?
>>>>
>>>> Yes, it should. You added -msse3 though, so are you asking about the
>>>> right thing?
>>>
>>> -msse3 is overly conservative, -msse2 would be enough.
>>>
>>> -Chris
>
>