[PATCH] D109123: [ARM][test] Add new tests for (mul (add r, c0), c1)

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 3 08:51:58 PDT 2021


RKSimon added inline comments.


================
Comment at: llvm/test/CodeGen/ARM/addimm-mulimm.ll:5
+
+define i32 @fold_addimm_mulimm_a(i32 %a) {
+; CHECK-LABEL: fold_addimm_mulimm_a:
----------------
Generally its better not to name tests sequentially like this, as it tends to get messy with later alterations. Maybe name them after the scale + offset imediates?

e..g fold_add19_mul11_i32 ?


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

https://reviews.llvm.org/D109123



More information about the llvm-commits mailing list