[all-commits] [llvm/llvm-project] 514381: [RISCV] Move hasOneUse() call after opcode check.

[Craig Topper via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 514381840c6d7aa775a092556992c87f022a361f
      https://github.com/llvm/llvm-project/commit/514381840c6d7aa775a092556992c87f022a361f
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2023-10-13 (Fri, 13 Oct 2023)

  Changed paths:
    M llvm/lib/Target/RISCV/RISCVISelLowering.cpp

  Log Message:
  -----------
  [RISCV] Move hasOneUse() call after opcode check.

hasOneUse can be more expensive for nodes with multiple outputs.
It's better to check the opcode first to skip nodes with multiple
outputs.

I have not seen an issue from this, just noticed while reviewing
code for a possible enhancement.