aaron.ballman added inline comments.
================
Comment at: clang/include/clang/AST/APValue.h:511-512
bool hasArrayFiller() const {
+ if (!isArray())
+ return false;
return getArrayInitializedElts() != getArraySize();
----------------
I think this makes the interface somewhat self-inconsistent. `hasLValuePath()`
asserts on `isLValue()`, and `getArraySize()` was providing a similar assertion
(on `isArray()`) when called from `hasArrayFiller()`.
I take that to mean that the expectation for this interface is that the caller
validates the `APValue` in the cases where it doesn't already know the answer.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D128248/new/
https://reviews.llvm.org/D128248
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
