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

[Chris Lattner]

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

:)

-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