[llvm-commits] [llvm] r120574 - /llvm/trunk/lib/Target/X86/X86ISelLowering.cpp

[Eric Christopher]

On Dec 1, 2010, at 4:58 AM, Duncan Sands wrote:

> Author: baldrick
> Date: Wed Dec  1 06:58:13 2010
> New Revision: 120574
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=120574&view=rev
> Log:
> I don't think it makes any sense to assert that the target supports SSE3 here.
> The user (i.e. whoever generated a call to the intrinsic in the first place) is
> essentially asking for a particular instruction to be placed in the assembler.
> If that instruction won't execute on the target machine, that's their problem
> not ours.  Two buildbots with processors that don't support SSE3 were barfing
> on the apm.ll test in CodeGen/X86 because of this assertion.

Or we could just make sure the testcase uses sse3 for codegen?

-eric