Revert "[OPENMP]Add support for analysis of if clauses."

This reverts commit rL366068.
The patch broke 86 tests under clang/test/OpenMP/ when run with address sanitizer.

llvm-svn: 366169

Author: alitamur

clang/include/clang/AST/OpenMPClause.h

@@ -501,10 +501,11 @@ class OMPIfClause : public OMPClause, public OMPClauseWithPreInit {
     return const_child_range(&Condition, &Condition + 1);
   }
 
-  child_range used_children();
+  child_range used_children() {
+    return child_range(child_iterator(), child_iterator());
+  }
   const_child_range used_children() const {
-    auto Children = const_cast<OMPIfClause *>(this)->used_children();
-    return const_child_range(Children.begin(), Children.end());
+    return const_child_range(const_child_iterator(), const_child_iterator());
   }
 
   static bool classof(const OMPClause *T) {

clang/lib/AST/OpenMPClause.cpp

@@ -209,25 +209,6 @@ const OMPClauseWithPostUpdate *OMPClauseWithPostUpdate::get(const OMPClause *C)
   return nullptr;
 }
 
-/// Gets the address of the original, non-captured, expression used in the
-/// clause as the preinitializer.
-static Stmt **getAddrOfExprAsWritten(Stmt *S) {
-  if (!S)
-    return nullptr;
-  if (auto *DS = dyn_cast<DeclStmt>(S)) {
-    assert(DS->isSingleDecl() && "Only single expression must be captured.");
-    if (auto *OED = dyn_cast<OMPCapturedExprDecl>(DS->getSingleDecl()))
-      return OED->getInitAddress();
-  }
-  return nullptr;
-}
-
-OMPClause::child_range OMPIfClause::used_children() {
-  if (Stmt **C = getAddrOfExprAsWritten(getPreInitStmt()))
-    return child_range(C, C + 1);
-  return child_range(&Condition, &Condition + 1);
-}
-
 OMPOrderedClause *OMPOrderedClause::Create(const ASTContext &C, Expr *Num,
                                            unsigned NumLoops,
                                            SourceLocation StartLoc,