[PATCH] D52117: Generate llvm.loop.parallel_accesses instead of llvm.mem.parallel_loop_access metadata.

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 18 17:38:21 PST 2018


hfinkel accepted this revision.
hfinkel added a comment.

Minor typo noted below, but otherwise, LGTM (to avoid any misunderstanding: this should be committed after the LLVM change lands).



================
Comment at: lib/CodeGen/CGLoopInfo.cpp:341
+    for (const LoopInfo &AL : Active) {
+      // Here we assume that ever loop that has an access group is parallel.
+      if (MDNode *Group = AL.getAccessGroup())
----------------
ever -> every


Repository:
  rC Clang

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

https://reviews.llvm.org/D52117





More information about the llvm-commits mailing list