[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