[PATCH] D77809: [Analyzer] Include typedef statements in CFG build.
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Apr 16 05:03:24 PDT 2020
aaron.ballman added inline comments.
================
Comment at: clang/test/Analysis/cfg.cpp:570
}
+// CHECK-LABEL: void vla_simple(int x)
----------------
Can you also add tests for:
```
int vla_unevaluated(int x) {
// Evaluates the ++x
sizeof(int[++x]);
// Does not evaluate the ++x
_Alignof(int[++x]);
_Generic((int(*)[++x])0, default : 0);
return x;
}
```
Also, it's a bit strange to find these VLA tests in a .cpp file. VLAs are a C++ extension and we should have some testing for them for constructs that are C++-specific, but I would have expected many of these tests to be in a C file instead.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D77809/new/
https://reviews.llvm.org/D77809
More information about the cfe-commits
mailing list