[PATCH] D88542: [ARM][LowOverheadLoops] Start insertion point

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 1 01:07:47 PDT 2020


SjoerdMeijer added inline comments.


================
Comment at: llvm/test/CodeGen/Thumb2/LowOverheadLoops/mov-operand.ll:32
 ; CHECK-NEXT:    vcvt.f32.u32 s4, s4
+; CHECK-NEXT:    dls lr, r4
 ; CHECK-NEXT:    vdiv.f32 s0, s0, s4
----------------
Maybe you can help me understanding this a bit better with this example. The DLS has been push down a bit, but I am guessing in this case it doesn't really matter?
Which kind of leads me to my more general question(s): is this an optimisation, or a bug fix, and which test case is the best to look at it?


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

https://reviews.llvm.org/D88542



More information about the llvm-commits mailing list