[PATCH] D47491: Expand constrained FP operations
Cameron McInally via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 11 11:19:13 PDT 2018
cameron.mcinally added a comment.
Thanks, guys.
I'd also like to point out that this Diff will slightly clobber Ulrich's https://reviews.llvm.org/D45576. But, I suspect that it will be a small change to accommodate the difference.
================
Comment at: test/CodeGen/X86/vector-constrained-fp-intrinsics.ll:5
+; CHECK-LABEL: constrained_vector_pow
+; COMMON: pow
+define <2 x double> @constrained_vector_pow() {
----------------
andrew.w.kaylor wrote:
> I think this check needs to do more than this. You should be verifying the complete expansion.
>
> Also, can you add checks for some other instructions. Some of these won't require expansion, right?
> I think this check needs to do more than this. You should be verifying the complete expansion.
Thanks. I've added a check for the upper POW.
> Also, can you add checks for some other instructions. Some of these won't require expansion, right?
POW is the only strict operation that I've found that currently needs expansion on X86. That said, I have a full set of tests that cover each strict operation. I could add them if you'd like, but it's a bit superfluous for this change. I do intend to add the full set once other patches are sent upstream. Thought?
Repository:
rL LLVM
https://reviews.llvm.org/D47491
More information about the llvm-commits
mailing list