[llvm-commits] [llvm] r139067 - /llvm/trunk/lib/Target/X86/X86InstrSSE.td

[Eric Christopher]

On Sep 6, 2011, at 10:59 AM, Bruno Cardoso Lopes wrote:

> This are all duplications of SSE patterns already in the .td file.
> None of the commits has added a new pattern for SSE instructions nor
> modified the behavior. I don't have testcases, this duplication is
> something I've done in mass since last year and I'm just fulfilling
> the holes, as to trying to compile and execute the llvm testsuite with
> AVX turned on (and make sure we don't have code size and performance
> regressions). If you feel this is not the way to go, I can revert all
> last 30 commits I've done, but I won't add testcases for all those
> right now. My future plan though is organize all sse tests in
> test/CodeGen/X86 (which are quite messy) and then duplicate all
> relevant ones to check for AVX versions instead.

Nah, sounds good to me. Should probably have more tests is what I'm saying.

-eric