[all-commits] [llvm/llvm-project] 5e9f17: [InstCombine] Add test for or(fshl(), fshl()) bswap...

[Simon Pilgrim via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 5e9f172295d011cc0a186a3e133b3b628556287f
      https://github.com/llvm/llvm-project/commit/5e9f172295d011cc0a186a3e133b3b628556287f
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2020-10-25 (Sun, 25 Oct 2020)

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

  Log Message:
  -----------
  [InstCombine] Add test for or(fshl(),fshl()) bswap pattern.

Currently InstCombinerImpl::matchBSwapOrBitReverse won't match starting from funnel shifts.


  Commit: 3052e474eceb25e3699940d6a7270550eed66606
      https://github.com/llvm/llvm-project/commit/3052e474eceb25e3699940d6a7270550eed66606
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2020-10-25 (Sun, 25 Oct 2020)

  Changed paths:
    M llvm/lib/Transforms/InstCombine/InstCombineAndOrXor.cpp
    M llvm/test/Transforms/InstCombine/bswap.ll

  Log Message:
  -----------
  [InstCombine] matchBSwapOrBitReversem - recognise or(fshl(),fshl()) bswap patterns.

I'm not certain InstCombinerImpl::matchBSwapOrBitReverse needs to filter the or(op0(),op1()) ops - there are just too many cases that recognizeBSwapOrBitReverseIdiom/collectBitParts handle now (and quickly).


Compare: https://github.com/llvm/llvm-project/compare/f81f09ba8950...3052e474eceb