[PATCH][AARCH64] Fixed fused multiply add/sub patterns

Jiangning Liu liujiangning1 at gmail.com
Sun Dec 22 23:14:55 PST 2013


Hi Ana,

The fix for fnmadd/fnmsub is OK for me.

But for single_use change, I'm not sure if the following claim is correct,
and it depends on hardware implementation, right?

> AArch64 targets most probably have one MAC pipe and this will be a
> performance issue as well.
>
So I think at least it would be better to get them decoupled and upstreamed
with two separate patches. If we find any mistake in future, it would be
easy for us to roll back as well. Also this optimization could probably
depends on different aarch64 target.

Thanks,
-Jiangning
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131223/83aa5bb7/attachment.html>


More information about the llvm-commits mailing list