[PATCH] D155270: [clang][Interp] Basic support for bit fields

[Timm Bäder via Phabricator via cfe-commits]

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