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

[Michael Kruse via Phabricator via llvm-commits]

Meinersbur added a subscriber: hfinkel.
Meinersbur added a comment.

OK, disregard that, I was looking at an old language reference (the first found by Google).

The part was added in by @hfinkel in rG411d31ad72456ba88c0b0bee0faba2b774add65f <https://reviews.llvm.org/rG411d31ad72456ba88c0b0bee0faba2b774add65f> and removed by me in rG978ba61536c2cdafa8454b7330c5d8e58d0d5048 <https://reviews.llvm.org/rG978ba61536c2cdafa8454b7330c5d8e58d0d5048> (might have been unintentional, at least don't remember removing intentionally)

As @pjaaskel noted in the bug report, the speculative semantics violate semantics such as by OpenMP. IMHO if really needed, speculation-safety should have its own metadata.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103907