[all-commits] [llvm/llvm-project] 6d1326: [RISCV] Add tests for combineBinOpOfZExts. NFC (#8...

[Luke Lau via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 6d13263d4a723689d025423562269ea6ccb6bfc2
      https://github.com/llvm/llvm-project/commit/6d13263d4a723689d025423562269ea6ccb6bfc2
  Author: Luke Lau <luke at igalia.com>
  Date:   2024-03-27 (Wed, 27 Mar 2024)

  Changed paths:
    A llvm/test/CodeGen/RISCV/rvv/binop-zext.ll

  Log Message:
  -----------
  [RISCV] Add tests for combineBinOpOfZExts. NFC (#86689)

Unlike add, sub and mul, we don't have widening instructions for div,
rem and logical ops, so we don't have any test coverage if we were to
extend combineBinOpOfZExts to handle them.

Adding tests coincidentally revealed that logical ops are already
narrowed as a generic DAG combine via
DAGCombiner::hoistLogicOpWithSameOpcodeHands. So we don't actually need
to run combineBinOpOfZExts on them.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications