[PATCH] D74970: [OpenMP] Refactor the analysis in checkMapClauseBaseExpression using StmtVisitor class.

Alexey Bataev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 21 14:11:49 PST 2020


ABataev added a comment.

In D74970#1887337 <https://reviews.llvm.org/D74970#1887337>, @cchen wrote:

> Fix test and sorry for my carelessness


No problem. But I don't see the changes in the test.



================
Comment at: clang/lib/Sema/SemaOpenMP.cpp:15473
+                         ERange);
+  if (Checker.Visit(E->IgnoreParenImpCasts())) {
+    return Checker.getFoundBase();
----------------
Remove braces here, the substatement is single line.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74970





More information about the cfe-commits mailing list