[PATCH] D150706: [LAA] Update MaxSafeDepDistBytes when non-unit stride

[Michael Maitland via Phabricator via llvm-commits]

michaelmaitland added a comment.

> I think there's no need to expose MaxSafeDepDistBytes outside of LAA, as users should use MaxSafeVectorWidthInBits WDYT?

I agree with this statement.

I think that the name of `MaxSafeDepDistBytes` should then to change to something such as `MinDepDistBytes` which represents the smallest dependence distance in bytes in the loop. In my opinion `MaxSafeDepDistBytes` according to its name and the semantics of its docstring *is* wrong without this patch. We could make the changes you and I discuss above, and abandon this patch. WDYT?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150706