[PATCH] D30086: Add generic IR vector reductions

[Michael Kuperstein via Phabricator via llvm-commits]

mkuper added inline comments.


================
Comment at: docs/LangRef.rst:11655
+reduction of a vector, returning the result as a scalar. The return type matches
+the element-type of the vector input.
+
----------------
mehdi_amini wrote:
> delena wrote:
> > mehdi_amini wrote:
> > > I think it'd be important to clarify the ordering: do we require a "fast" FMF to change it? If not, why?
> > I think that in the case of FP operations we need two intrinsics - "ordered" and "fast".
> Why don't reusing the existing FMF? The intrinsic without FMF would be equivalent to "ordered" and the "fast" FMF flag would allow any ordering.
I don't think we can currently attach FMF to CallInsts.


Repository:
  rL LLVM

https://reviews.llvm.org/D30086