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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
