[PATCH] D158291: [PoC][WIP] Add an AArch64 specific pass for loop idiom recognition

David Sherwood via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 13 08:47:06 PST 2023


david-arm added a comment.

In D158291#4652947 <https://reviews.llvm.org/D158291#4652947>, @craig.topper wrote:

> Does the new pass need to check that SVE is enabled before doing the transform?

Hi @craig.topper, good point! I've added a check that the target supports scalable vectors and that we know the minimum page size. Although the pass currently lives in lib/Target/AArch64 it is generic enough that it could be moved into a common directory and used by other targets.

It might make sense to move this patch into github soon for a full review, even though I prefer Phabricator. :)


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

https://reviews.llvm.org/D158291



More information about the llvm-commits mailing list