[PATCH] D130418: [LoopDataPrefetch] Fix crash when TTI doesn't set CacheLineSize
Allen zhong via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 25 04:22:12 PDT 2022
Allen added a comment.
In D130418#3675649 <https://reviews.llvm.org/D130418#3675649>, @tschuett wrote:
> Do you force all targets to provide `PrefetchDistance` and `PrefetchDistance` or could it be an `llvm::optional` ?
Thanks, I think it is unfriendly to crash at the moment. Use a default value when they are not specified can be independent next step, if you feel the need.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D130418/new/
https://reviews.llvm.org/D130418
More information about the llvm-commits
mailing list