[PATCH] D86841: [clang] Add mustprogress and llvm.loop.mustprogress attribute deduction

Johannes Doerfert via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 2 17:18:43 PST 2020


jdoerfert accepted this revision.
jdoerfert added a comment.
This revision is now accepted and ready to land.

LGTM, some nits



================
Comment at: clang/lib/CodeGen/CGStmt.cpp:801
+  } else if (LanguageRequiresProgress())
+    LoopMustProgress = true;
+
----------------
Nit: Add braces here.


================
Comment at: clang/lib/CodeGen/CGStmt.cpp:894
+  } else if (LanguageRequiresProgress())
+    LoopMustProgress = true;
+
----------------
Same as above.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86841



More information about the cfe-commits mailing list