[all-commits] [llvm/llvm-project] 913229: [LegalizeTypes][VE] Don't Expand BITREVERSE/BSWAP ...

[Craig Topper via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 913229983633cd4c19b9e5534018f9a42e274b30
      https://github.com/llvm/llvm-project/commit/913229983633cd4c19b9e5534018f9a42e274b30
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2021-06-29 (Tue, 29 Jun 2021)

  Changed paths:
    M llvm/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp
    M llvm/test/CodeGen/VE/Scalar/bitreverse.ll

  Log Message:
  -----------
  [LegalizeTypes][VE] Don't Expand BITREVERSE/BSWAP during type legalization promotion if they will be promoted for NVT in op legalization.

We were trying to expand these if they were going to be expanded
in op legalization so that we generated the minimum number of
operations. We failed to take into account that NVT could be
promoted to another legal type in op legalization.

Hoping this fixes the issue on the VE target reported as a follow
up to D96681. The check line changes were taken from before
1e46b6f4012399a2fef5fbbb4ed06fc919835414 so this patch does
appear to improve some cases that had previously regressed.