[PATCH] D53513: [OPENMP] Add support for 'atomic_default_mem_order' clause on requires directive

Patrick Lyster via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 22 12:14:15 PDT 2018


patricklyster added inline comments.


================
Comment at: clang/include/clang/AST/OpenMPClause.h:930
+  /// Returns location of clause kind.
+  SourceLocation getAtomicDefaultMemOrderKindKwLoc() const { return KindKwLoc; }
+
----------------
ABataev wrote:
> Again, probably too long line.
None of these lines are longer than 80 symbols - hard to see this on Phabricator though. All this code was formatted using the clang-format tool. Would you still like me to change it?


Repository:
  rC Clang

https://reviews.llvm.org/D53513





More information about the cfe-commits mailing list