SebastianPerta added a comment.
>> Regarding testing, it seems like this should change the diagnostic emitted
>> for long x = 'abc' from "Character constant too long for its type" to
>> "Multi-character character constant" when the target is AVR?
Currently on RL78 it returns this warning without any extra changes:
warning: multi-character character constant [-Wmultichar]
long x = 'abc';
^
@sammccall this is what you expect, correct?
I haven't built LLVM for AVR yet, I will do it and get back to you.
>> I think this should have a test, and we can probably use _Static_assert for
>> it: https://godbolt.org/z/611bx47qc
Thank you for the help @aaron.ballman , I will update the patch with a test
case using _Static_assert.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D127363/new/
https://reviews.llvm.org/D127363
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
