tbaeder added a comment.

In D155270#4550466 <https://reviews.llvm.org/D155270#4550466>, @aaron.ballman 
wrote:

> Precommit CI is currently crashing on the newly introduced test case.

It's missing https://reviews.llvm.org/D155548 :)

That's of course just a workaround. At the beginning of the `evalute*` fuctions 
in the new interpreter, there's a `assert(S.Stk.empty())`, to ensure that a 
previous interpret call didn't leave someting on the stack. But that doesn't 
work anymore since `FieldDecl::getBitWidthValue()` calls in the interpreter 
again while we're already evaluating something, so that assertion triggers.


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

https://reviews.llvm.org/D155270

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

Reply via email to