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

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 16 10:23:51 PDT 2018


spatel added a comment.

For reference, let me repeat the next steps assuming we're close to good here (I just wrote this in https://reviews.llvm.org/D47735 ):

1. Convert the intrinsics directly to ROTL/ROTR nodes (this should be a ~2 line patch in SelectionDAGBuilder).
2. Expose the intrinsic as clang or other front-end builtins (the first of these will be __builtin_rotate* rather than the more general __builtin_funnel_shift).
3. Add simplifications/folds/analysis for the intrinsics to IR passes.
4. Canonicalize to the intrinsics in instcombine.

Let me know if I'm missing anything.


https://reviews.llvm.org/D49242





More information about the llvm-commits mailing list