[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