[PATCH] D106408: Allow rematerialization of virtual reg uses

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 5 01:21:12 PDT 2021


dmgreen added a comment.

Thanks. This sounds OK for ARM to me.



================
Comment at: llvm/test/CodeGen/Thumb2/ldr-str-imm12.ll:52
+; CHECK: strd r{{[0-9]+}}, {{(r[0-9]+)|(lr)}}, [sp
+; CHECK: strd r{{[0-9]+}}, {{(r[0-9]+)|(lr)}}, [sp
   store volatile %union.rec* null, %union.rec** @zz_hold, align 4
----------------
On second thoughts, can you just run the update_llc_test_checks on this file?

The strd is just two adjacent str glommed together, so this is checking for more stores than it was before. But there appear to be extra stores in the original, including stm.  From the comment at the top of the file it is not very clear what this is trying to test now. Just updating the test checks so that we see the whole function sounds like the best idea.


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

https://reviews.llvm.org/D106408



More information about the llvm-commits mailing list