[PATCH] D95672: [LV] Vectorize backward dependence

Hu Jiangping via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 11 03:03:58 PST 2021


hujp added a comment.

In D95672#2617780 <https://reviews.llvm.org/D95672#2617780>, @fhahn wrote:

> 2. Does the re-order map generation need to happen in LVL or could it be part of LoopAccessInfo, like the other memory checks? That way, other clients could also benefit.

Hi fhahn! Thank you for your review. I'll consider your questions and comments carefully.
Firstly, for the No.2, I think we can do the checks after `MemoryDepChecker::isDependent` returned `Dependence::Backward` ,and extend the `MemoryDepChecker::Dependence` struct to save the reorder list. What do you think about?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95672



More information about the llvm-commits mailing list