[PATCH] D22267: [AArch64] Properly validate the reciprocal estimation

Evandro Menezes via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 12 08:50:03 PDT 2016


evandro added a comment.

@t.p.northover,

WRT the tests, I'd appreciate some help.  In some cases, lowering would fail because the square root was being expanded into the series for MVT::v4f64, which is obviously not supported by AArch64.  I'm at a loss as to why such data type, unsupported by the target, percolated through to the backend.

Would concocting the bytecode using such data type be a sensible test?

On another note, as I'm not very familiar with the latest versions of ARMv8, should FP16 be considered here?

Thank you.


Repository:
  rL LLVM

http://reviews.llvm.org/D22267





More information about the llvm-commits mailing list