[clang] [clang] Assert the enum FPOpts and LangOpts fit into the storage (PR #126166)

Zahira Ammarguellat via cfe-commits cfe-commits at lists.llvm.org
Mon Feb 10 12:12:35 PST 2025


zahiraam wrote:

> > Not sure I understand your question? Which assertion? `clang.exe t.c` is triggering an assertion?
> 
> I don't understand why you say
> 
> > The value `CX_None` is the defaut value given to the `Range` when no `fcomplex-arithmetic` is used on the command lin
> 
> but imply it does not need to be stored in the bitfield. If you patch this CL, and change the size for `ComplexRange` back, you will see it that it fails the assertion I added if you run the clang test suite.

OK. I see the issue. Thanks.
I think this should be tilte [NFC][Clang] YOUR TITLE.
If you don't want to make it NFC then you need to add a LIT test :-) Not sure you can make it fail with a LIT test.

https://github.com/llvm/llvm-project/pull/126166


More information about the cfe-commits mailing list