[PATCH] D71225: [OpenMP][WIP] atomic update only evaluate once for expression having side effect

Alexey Bataev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 9 13:49:53 PST 2019


ABataev added a comment.

It is against the standard, I assume. According to standard, `During the execution of an atomic region, multiple syntactic occurrences of x must designate the same storage location.`. In your case, this is not so. I assume it would be better to implement the diagnostic here that the code is not OpenMP compliant.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71225





More information about the cfe-commits mailing list