[PATCH] D147750: [LAA/LV] Simplify stride speculation logic [nfc]

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 8 10:03:29 PDT 2023


fhahn added a reviewer: Ayal.
fhahn added a comment.

LGTM as this simplifies things slightly by avoiding a Value <-> SCEV roundtrip. Please wait a day or so with committing in case other reviewers have additional thoughts.



================
Comment at: llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp:3407
                                        PredicatedScalarEvolution *PSE) {
-  const ValueToValueMap &Strides = ValueToValueMap();
+  const DenseMap<Value *, const SCEV *> &Strides =
+    DenseMap<Value *, const SCEV *>();
----------------
nit: could just be `auto &`, given hat the type is already clear from the object construction.


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

https://reviews.llvm.org/D147750



More information about the llvm-commits mailing list