[PATCH] D103907: [LV] Parallel annotated loop does not imply all loads can be hoisted.

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 10 13:54:36 PDT 2021


jdoerfert added a comment.

FWIW, if someone adds documentation what `#pragma clang loop vectorize(assume_safety)` actually means (somewhere), we can go back and make it imply if-conversion. However, we should not conflate it with the access groups/parallel metadata. Instead, annotate loads as "speculatable" (or sth. similar) and we go with that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103907



More information about the llvm-commits mailing list