[PATCH] DAGCombiner: Continue combining if FoldConstantArithmetic() fails.
matze at braunis.de
Tue Feb 3 14:30:45 PST 2015
New version: Turns out if you continue combining in the presence of OpaqueConstants you have to be very carefull to not accidentally fold them into non-Opaque ones during a combine. This patch carefully finds these cases now (I basically looked at all places that create new target constants and checked whether they would be harmfully replace OpaqueConstants with new ones). I did extensive x86 test-suite runs and couldn't find any performance differences.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 15976 bytes
Desc: not available
More information about the llvm-commits