[PATCH] D73064: [LoopCacheAnalysis]: Add support for negative stride

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added inline comments.


================
Comment at: llvm/test/Analysis/LoopCacheAnalysis/PowerPC/compute-cost.ll:70
+
+; CHECK: Loop 'for.cond2' has cost = 64
+define void @Test2(%struct._Handleitem** %B) {
----------------
As I mentioned, this should *not* be scored as the ones below. Add a FIXME here to explain that. In the code we should also add a FIXME to indicate that this is not the best handling of negative strides.


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

https://reviews.llvm.org/D73064