[all-commits] [llvm/llvm-project] 186d5c: [RISCV] Make getInstSeqCost handle other Zb* instr...

Craig Topper via All-commits all-commits at lists.llvm.org
Wed Apr 20 22:08:45 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 186d5c8af5829bc4f6f0a3edcca43cef2d498c26
      https://github.com/llvm/llvm-project/commit/186d5c8af5829bc4f6f0a3edcca43cef2d498c26
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2022-04-20 (Wed, 20 Apr 2022)

  Changed paths:
    M llvm/lib/Target/RISCV/MCTargetDesc/RISCVMatInt.cpp

  Log Message:
  -----------
  [RISCV] Make getInstSeqCost handle other Zb* instructions.

We haven't been updating this as Zb* instructions have been used
for immediate materialization. They will hit the default case and
trigger an llvm_unreachable. Instead of trying to list them all,
assume instructions that aren't explicitly listed aren't compressible.

Spotted while looking at integer materialization for other reasons.
I haven't seen a crash from this yet.




More information about the All-commits mailing list