[PATCH] D83159: [RISCV] Add a new codegen test

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 6 17:55:19 PDT 2020


MaskRay added a comment.

Optional: you can add `[test]` to the subject, i.e. `[RISCV][test]` to make it clear this patch is completely about tests. This is even stronger than `NFC` (non-functional change).



================
Comment at: llvm/test/CodeGen/RISCV/addimm-mulimm.ll:1
+; RUN: llc -mtriple=riscv32 -mattr=+m -verify-machineinstrs < %s \
+; RUN:   | FileCheck -check-prefix=RV32IM %s
----------------
It'd be better adding a file-level comment.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83159





More information about the llvm-commits mailing list