[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).
More information about the llvm-commits