[PATCH] D92105: [RISCV] Add pre-emit pass to make more instructions compressible

Jessica Clarke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 11 09:59:09 PST 2020


jrtc27 added inline comments.


================
Comment at: llvm/test/CodeGen/RISCV/compress-instrs.mir:2
+# RUN: llc -o - %s -mtriple=riscv32 -mattr=+c -simplify-mir \
+# RUN: -run-pass=riscv-compress-instrs | FileCheck %s
+# RUN: llc -o - %s -mtriple=riscv64 -mattr=+c -simplify-mir \
----------------
jrtc27 wrote:
> Please indent continuation lines with an additional 4 spaces.
Or maybe the style is just 2. For `|` it seems to be generally 2, but for arguments that wrap it seems to be any of 2, 3 or 4 depending on where you look.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D92105/new/

https://reviews.llvm.org/D92105



More information about the llvm-commits mailing list