smeenai added a comment.

In D131528#3723231 <https://reviews.llvm.org/D131528#3723231>, @ayermolo wrote:

> In D131528#3722395 <https://reviews.llvm.org/D131528#3722395>, @shafik wrote:
>
>> In D131528#3719430 <https://reviews.llvm.org/D131528#3719430>, @ayermolo 
>> wrote:
>>
>>> Was this and previous change intended to capture this case?
>>>  const enum NumberType neg_one = (enum NumberType) ((enum NumberType) 0 - 
>>> (enum NumberType) 1);
>>
>> That was not intended and I plan on fixing that.
>
> Ah I see. Thanks for elaborating. Can you tag me in the fix diff please.

D131874 <https://reviews.llvm.org/D131874> seems like the fix for that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131528

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

Reply via email to