samparker added inline comments.

================
Comment at: llvm/lib/Target/ARM/ARMLowOverheadLoops.cpp:1250
   else
-    return false;
+    return Changed;
 
----------------
dblaikie wrote:
> Also might be worth reaching out to authors to check that this change is 
> intended & possibly tested.
Ah, yes this should be fine. The Changed value would only be used by the 
pipeline manager and this pass is the last transform in the Arm pipeline.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102942

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to