[PATCH] D47491: Expand constrained FP operations
Cameron McInally via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed May 30 11:34:16 PDT 2018
cameron.mcinally added a comment.
In https://reviews.llvm.org/D47491#1116472, @cameron.mcinally wrote:
> Updated this patch to handle a constrained vector POWI. The arguments to that function do not have a uniform type.
>
> I have not included a test case for POWI since the intrinsic declaration also needs to be updated to handle vectors. That is a separate issue and probably deserves its own Diff.
The LLVM LangRef states that the POWI intrinsic always takes a scalar i32 as the 2nd argument, even when the 1st argument is a vector. That seems wrong, but I'll save the llvm-dev discussion for another time.
I'll have to special case the POWI expansion in light of this...
https://reviews.llvm.org/D47491
More information about the llvm-commits
mailing list