[PATCH] D76709: [Target][ARM] Adding MVE VPT Optimisation Pass
Pierre van Houtryve via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Apr 3 00:30:08 PDT 2020
Pierre-vh marked 2 inline comments as done.
Pierre-vh added inline comments.
================
Comment at: llvm/lib/Target/ARM/MVEVPTOptimisationsPass.cpp:192
+ // remove the kill flag.
+ if (PrevVCMPResultKiller)
+ PrevVCMPResultKiller->setIsKill(false);
----------------
dmgreen wrote:
> Should we be clearing this value at some points? Setting it back to nullptr?
Sure, it should be cleared after. I'll see if I can add a test for that as well.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D76709/new/
https://reviews.llvm.org/D76709
More information about the llvm-commits
mailing list