[Lldb-commits] [PATCH] D134493: [lldb][TypeSystemClang] Deduce lldb::eEncodingUint for unsigned enum types

Fri Sep 23 02:55:59 PDT 2022

labath added a comment.

In D134493#3810290 <https://reviews.llvm.org/D134493#3810290>, @Michael137 wrote:

> Wasn't sure how to properly test this since the original reproducer technically relies on implementation-defined behaviour (i.e., initialising a bitfield with an out-of-range value). Suggestions are welcome

I'm probably missing something, but what exactly is undefined about that test program? The number eight fits comfortably in four bits, and afaik it is a valid value for the `EnumVals` type because it has a fixed underlying type.

Other than that, this seems fine to me.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134493