[PATCH] Implement aarch64 neon instruction class AdvSIMD (by element) - Clang
Tim Northover
t.p.northover at gmail.com
Mon Sep 30 02:31:57 PDT 2013
================
Comment at: utils/TableGen/NeonEmitter.cpp:1553-1554
@@ -1517,3 +1552,4 @@
case OpMlaLane:
+ case OpFMALane:
s += "__a + (__b * " + SplatLane(nElts, "__c", "__d") + ");";
break;
----------------
Sorry, I should have spotted this earlier, but I think that an fma is semantically distinct from the C expression "a+b*c".
The latter we *can* transform into an fma in -ffast-math mode, but not generally. So we probably need an intrinsic to make sure that if the user asks for a fused multiply, that's the operation they get.
Actually, there'a already a generic llvm intrinsic to handle this: "@llvm.fma.*"; we should probably use that rather than an ARM/AArch64-specific one.
http://llvm-reviews.chandlerc.com/D1752
More information about the cfe-commits
mailing list