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

[Chris Lattner]

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