[PATCH] D128248: [clang] Avoid an assertion in APValue::hasArrayFiller()

Shafik Yaghmour via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 29 13:21:45 PDT 2022


shafik added a comment.

> and run the test case, the first added line prints `1` and the second one `0`. `Result` is being mutated when doing the in-place evaluation.

I did not catch that. That is unfortunate, I am wondering now if we need a `Result.isArray() && ` before the `EvaluateInPlace` in that loop.


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

https://reviews.llvm.org/D128248



More information about the cfe-commits mailing list