[all-commits] [llvm/llvm-project] 87f3eb: [InstCombine] add test for bogus bswap; NFC

[Sanjay Patel via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 87f3ebd505864a4a28c476fa63fc38a334bec1f9
      https://github.com/llvm/llvm-project/commit/87f3ebd505864a4a28c476fa63fc38a334bec1f9
  Author: Sanjay Patel <spatel at rotateright.com>
  Date:   2022-03-23 (Wed, 23 Mar 2022)

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

  Log Message:
  -----------
  [InstCombine] add test for bogus bswap; NFC

This is reduced from a crash caused by D122166.


  Commit: 0fcff69bcb3d589e1feef4cd4902a2b48b6c7435
      https://github.com/llvm/llvm-project/commit/0fcff69bcb3d589e1feef4cd4902a2b48b6c7435
  Author: Sanjay Patel <spatel at rotateright.com>
  Date:   2022-03-23 (Wed, 23 Mar 2022)

  Changed paths:
    M llvm/lib/Transforms/InstCombine/InstCombineShifts.cpp
    M llvm/test/Transforms/InstCombine/lshr.ll

  Log Message:
  -----------
  [InstCombine] try to narrow shifted bswap-of-zext (2nd try)

The first attempt at this missed a validity check.
This version includes a test of the narrow source
type for modulo-16-bits.

Original commit message:

This is the IR counterpart to 370ebc9d9a573d6
which provided a bswap narrowing fix for issue #53867.

Here we can be more general (although I'm not sure yet
what would happen for illegal types in codegen - too
rare to worry about?):
https://alive2.llvm.org/ce/z/3-CPfo

This will be more effective if we have moved the shift
after the bswap as proposed in D122010, but it is
independent of that patch.

Differential Revision: https://reviews.llvm.org/D122166


Compare: https://github.com/llvm/llvm-project/compare/4e88cb6825ee...0fcff69bcb3d