[PATCH] D90024: [ARM][SchedModels] Get rid of IsLdrAm2ScaledPred

Andrea Di Biagio via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 23 08:47:52 PDT 2020


andreadb added a comment.

In D90024#2350389 <https://reviews.llvm.org/D90024#2350389>, @dmgreen wrote:

> Should it always have been checking operand 4 then? I think that makes sense
>
> (More of a commit message might make it more obvious why changes are being made ;) )

I was about to ask something similar...
The original predicate is checking operand 3, not 4. I am a bit confused.


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

https://reviews.llvm.org/D90024



More information about the llvm-commits mailing list