[PATCH] D73638: [AST] Move dependence computations into a separate file

Diogo N. Sampaio via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 17 02:17:34 PDT 2020


dnsampaio added inline comments.


================
Comment at: clang/lib/AST/ComputeDependence.cpp:607
+  auto D = toExprDependence(E->getType()->getDependence());
+  for (auto *E : E->arguments())
+    D |= E->getDependence() & ~ExprDependence::Type;
----------------
I'm impressed this even compiled. With gcc 8.3 I get an error:
```
/work/bf/LLVM/src/clang/lib/AST/ComputeDependence.cpp:607:18: error: use of ‘E’ before deduction of ‘auto’
   for (auto *E : E->arguments())
                  ^
```


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73638





More information about the cfe-commits mailing list