[all-commits] [llvm/llvm-project] ef1ccc: [RISCV] Move comments before 'if' instead of after...

Craig Topper via All-commits all-commits at lists.llvm.org
Sat Jul 15 22:54:01 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: ef1ccc493e6167488ac10da2842fa7cac2746565
      https://github.com/llvm/llvm-project/commit/ef1ccc493e6167488ac10da2842fa7cac2746565
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2023-07-15 (Sat, 15 Jul 2023)

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

  Log Message:
  -----------
  [RISCV] Move comments before 'if' instead of after. NFC

This allows us to remove some curly braces around the if body.
The code wasn't consistent about it anyway. Comments before is
used in other places in this file already.

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


  Commit: d09109aa1ec104b51ce7c2b06ecbc0c52fe2f780
      https://github.com/llvm/llvm-project/commit/d09109aa1ec104b51ce7c2b06ecbc0c52fe2f780
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2023-07-15 (Sat, 15 Jul 2023)

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

  Log Message:
  -----------
  [RISCV] Use isScalarInteger instead of isInteger. NFC

The type should only be scalar here and the isScalarInteger
should be a simpler check.


Compare: https://github.com/llvm/llvm-project/compare/2b0b85c05e74...d09109aa1ec1


More information about the All-commits mailing list