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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to