[PATCH] D49242: [Intrinsics] define funnel shift IR intrinsics + DAG builder support

[Sanjay Patel via Phabricator via llvm-commits]

spatel added inline comments.


================
Comment at: docs/LangRef.rst:11915
+vector with integer element type. All arguments and the return value must
+have the same type.
+
----------------
RKSimon wrote:
> Is the shift amount going to be treated as a modulo value?
Yes - at least that's what I think we want. It's the last line under 'Semantics' because this Overview/Arguments/Semantics organization is what was copy/pasted from the existing intrinsic docs. 

Let me move/repeat that bit, so we state that from the beginning.

And I'll make a docs build target as suggested to avoid formatting bugs.


https://reviews.llvm.org/D49242