[PATCH] D102942: Remove or use variables which are unused but set.
Sam Parker via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 1 05:27:00 PDT 2021
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
More information about the llvm-commits
mailing list