[PATCH] Resolve issues with fmuladd intrinsic handling across multiple backends

Stephen Lin swlin at post.harvard.edu
Tue Jul 9 09:44:46 PDT 2013


> I'm not sure this is what you want.  Graphics-oriented users sometimes have weird vector sizes (<3 x float>, <3 x half>, etc.) that should still get fused if possible.  I think you should be able to implement the checks you need without VT having to be simple.
>
> Otherwise, your SelectionDAG changes look fine.
>

Hi Owen, in addition to allowing extended types I changed DAGCombiner
to use the LegalOperations boolean to detect whether or not it's safe
to introduce in illegal operation, rather than introducing my own.

Can you take a look at the new patch and make sure I didn't do
anything silly, if you get a chance? Basically, as I understand it,
LegalOperations is true if we've already legalized all operations and
false otherwise.

Thanks,
Stephen
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fma-faster-check-types.patch
Type: application/octet-stream
Size: 26781 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130709/dbe51b6a/attachment.obj>


More information about the llvm-commits mailing list