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

Shafik Yaghmour via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 27 14:09:01 PDT 2022


shafik added a comment.

Are you sure it is failing on the line you indicated? After looking at the code, a little before that line we have:

  Result = APValue(APValue::UninitArray(), NumEltsToInit, NumElts);

which should assure that we have an array at this point. Looking at the result of the code prior to this they evaluate the check for `Result.isArray()` and so we should be consistent.


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

https://reviews.llvm.org/D128248



More information about the cfe-commits mailing list