[PATCH] D144218: [Clang] [AVR] Fix USHRT_MAX for 16-bit int.

Sergei Barannikov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 10 10:08:59 PDT 2023


barannikov88 added a comment.

In D144218#4255686 <https://reviews.llvm.org/D144218#4255686>, @aaron.ballman wrote:

> In D144218#4254021 <https://reviews.llvm.org/D144218#4254021>, @barannikov88 wrote:
>
>> There is something wrong with the `#if`s in the test, isn't it? They all look no-op.
>
> Those are verifying that the macro expands to a value that can be used by the preprocessor to avoid issues like the one pointed out in this comment: https://reviews.llvm.org/D144218#inline-1396599

Ah, I see, thanks. I should have read the comments more carefully.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144218



More information about the cfe-commits mailing list