[PATCH] D72241: [clang-tidy] new altera single work item barrier check

Eugene Zelenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Jan 5 21:59:33 PST 2020


Eugene.Zelenko added a comment.

I think will be good idea to separate module code in own review or refer to previous one of previous reviews as dependency.



================
Comment at: clang-tidy/altera/SingleWorkItemBarrierCheck.cpp:51
+    if (Attribute->getKind() == attr::Kind::ReqdWorkGroupSize) {
+      auto RWGSAttribute =
+          static_cast<const ReqdWorkGroupSizeAttr *>(Attribute);
----------------
const auto *.


================
Comment at: clang-tidy/altera/SingleWorkItemBarrierCheck.cpp:69
+  } else {
+    if (IsNDRange == true) { // No warning if kernel is treated as an NDRange.
+      return;
----------------
Just IsNDRange. See readability-simplify-boolean-expr.


================
Comment at: docs/clang-tidy/checks/altera-single-work-item-barrier.rst:51
+
+   Defines the version of the Altera Offline Compiler. Defaults to 1600
+   (corresponding to version 16.00).
----------------
Please highlight 1600 with single back-quotes.


Repository:
  rCTE Clang Tools Extra

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

https://reviews.llvm.org/D72241





More information about the cfe-commits mailing list