[llvm-commits] PATCH: Fix for possible bug in DAGCombiner: visitSTORE

[Tom Stellard]

Hi Duncan,

The DAGCombiner was generating some illegal stores (f32 stores, which
I have marked as custom) for the R600 backend, and I uncovered what I
think is a bug in the visitSTORE function.  I have attached a patch that
fixes the bug on R600, but it causes 4 X86 tests to fail:

LLVM :: CodeGen/X86/2008-06-13-NotVolatileLoadStore.ll
LLVM :: CodeGen/X86/2008-06-13-VolatileLoadStore.ll
LLVM :: CodeGen/X86/mmx-arg-passing.ll
LLVM :: CodeGen/X86/vec_extract-sse4.ll

You were the last person to modify this piece of code (in 2008!), so I
was wondering if you could take a look.

Thanks,
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Fix-possible-bug-in-DAGCombiner-visitSTORE.patch
Type: text/x-diff
Size: 917 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20121206/960c8c57/attachment.patch>