[all-commits] [llvm/llvm-project] 4e6adb: [RISCV] Transform fixable instruction in place in ...

Craig Topper via All-commits all-commits at lists.llvm.org
Mon Nov 21 19:27:10 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 4e6adb394ebcdb2350fac6b47f7b789750e1f300
      https://github.com/llvm/llvm-project/commit/4e6adb394ebcdb2350fac6b47f7b789750e1f300
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2022-11-21 (Mon, 21 Nov 2022)

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

  Log Message:
  -----------
  [RISCV] Transform fixable instruction in place in RISCVSExtWRemoval. NFC

Instead of creating a new instruction and copying operands, we can
use setDesc to convert in place.

Reviewed By: reames

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


  Commit: f99514ac3a6fbf8af619f45a93a22c81bb7f9e22
      https://github.com/llvm/llvm-project/commit/f99514ac3a6fbf8af619f45a93a22c81bb7f9e22
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2022-11-21 (Mon, 21 Nov 2022)

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

  Log Message:
  -----------
  [RISCV] Remove SExtWRemovalCands set from RISCVSExtWRemoval.

After D137970, we do the fixable instruction conversion in place
so we don't need to worry about iterator invalidation. This lets
us to conversion and updates in a single loop.

Reviewed By: reames

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


Compare: https://github.com/llvm/llvm-project/compare/a391b49ce852...f99514ac3a6f


More information about the All-commits mailing list