SjoerdMeijer added inline comments.

================
Comment at: include/clang/Basic/arm_neon.td:1504
+  // Scalar floating point multiply extended (scalar, by element)
+  def SCALAR_FMULX_LANEH : IOpInst<"vmulx_lane", "ssdi", "Sh", 
OP_SCALAR_MUL_LN>;
+  def SCALAR_FMULX_LANEQH : IOpInst<"vmulx_laneq", "ssji", "Sh", 
OP_SCALAR_MUL_LN>;
----------------
I found that unfortunately it's not that straightforward. This leads to wrong 
code generation as it is generating a fmul instead of fmulx. I am suspecting 
this instruction description should be using OP_SCALAR_MULX_LN, but also the 
type decls are wrong. Need to dig a bit further here.


https://reviews.llvm.org/D44222



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to