================
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
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to