[all-commits] [llvm/llvm-project] 629e1e: [InstCombine] add tests for funnel-shift with shuf...

Sanjay Patel via All-commits all-commits at lists.llvm.org
Wed May 4 10:07:45 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 629e1e82e45864fd959d3abfa2ac77eb7b9e180f
      https://github.com/llvm/llvm-project/commit/629e1e82e45864fd959d3abfa2ac77eb7b9e180f
  Author: Sanjay Patel <spatel at rotateright.com>
  Date:   2022-05-04 (Wed, 04 May 2022)

  Changed paths:
    M llvm/test/Transforms/InstCombine/fsh.ll

  Log Message:
  -----------
  [InstCombine] add tests for funnel-shift with shuffled operands; NFC


  Commit: 7e6d318c506bbbffb4f92cd8b70773beedc78c3e
      https://github.com/llvm/llvm-project/commit/7e6d318c506bbbffb4f92cd8b70773beedc78c3e
  Author: Sanjay Patel <spatel at rotateright.com>
  Date:   2022-05-04 (Wed, 04 May 2022)

  Changed paths:
    M llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp
    M llvm/test/Transforms/InstCombine/fsh.ll

  Log Message:
  -----------
  [InstCombine] move shuffle after funnel shift with same-shuffled operands

This extends 432c199e8473 and 9c4770eaab9d9 with an intrinsic
cited directly in issue #46238

Eventually, we will want to use llvm::isTriviallyVectorizable()
or create some new API for this list, but for now, I am intentionally
making a minimum change to reduce risk and only affect an intrinsic
with regression tests in place.


  Commit: 14f257620c53f2767458c56e42a42f31ca32751d
      https://github.com/llvm/llvm-project/commit/14f257620c53f2767458c56e42a42f31ca32751d
  Author: Sanjay Patel <spatel at rotateright.com>
  Date:   2022-05-04 (Wed, 04 May 2022)

  Changed paths:
    M llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp
    M llvm/test/Transforms/InstCombine/fsh.ll

  Log Message:
  -----------
  [InstCombine] add type constraint to intrinsic+shuffle fold

This check is in the related fold for binops,
but it was missed when the code was adapted
for intrinsics in 432c199e8473. The new test
would crash when trying to create a new
intrinsic with mismatched types.


Compare: https://github.com/llvm/llvm-project/compare/62501bc45a2f...14f257620c53


More information about the All-commits mailing list