[PATCH] D21949: [SystemZ] Utilize Test Data Class instructions.

Marcin Koƛcielnicki via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 4 11:41:49 PDT 2016


koriakin added a comment.

In http://reviews.llvm.org/D21949#473812, @uweigand wrote:

> I haven't looked into the detailed implementation yet.  However, one important feature to verify would be that use of the TEST DATA CLASS instruction is not only triggered by those specific IR sequences you test for on the LLVM level, but more importantly, triggered by whatever code clang generates for the isinf/isnan/signbit/fpclassify etc. builtins, since those are what users (hopefully) typically use ...  [ This might actually involve changing clang as well, not sure about that.  ]


I've covered signbit and isnormal in the tests, I'll update it to cover isinf and fpclassify as well.  isnan on its own is not profittable to implement with TEST DATA CLASS (a normal compare is good enough for that).


Repository:
  rL LLVM

http://reviews.llvm.org/D21949





More information about the llvm-commits mailing list