[PATCH] D144218: [Clang] [AVR] Fix USHRT_MAX for 16-bit int.
Daniel Thornburgh via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Feb 16 14:12:35 PST 2023
mysterymath created this revision.
mysterymath added reviewers: aaron.ballman, aykevl, dylanmckay.
Herald added a subscriber: Jim.
Herald added a project: All.
mysterymath requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
For AVR, the definition of USHRT_MAX overflows.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D144218
Files:
clang/lib/Headers/limits.h
Index: clang/lib/Headers/limits.h
===================================================================
--- clang/lib/Headers/limits.h
+++ clang/lib/Headers/limits.h
@@ -52,7 +52,7 @@
#define LONG_MIN (-__LONG_MAX__ -1L)
#define UCHAR_MAX (__SCHAR_MAX__*2 +1)
-#define USHRT_MAX (__SHRT_MAX__ *2 +1)
+#define USHRT_MAX (__SHRT_MAX__ * 2U + 1U)
#define UINT_MAX (__INT_MAX__ *2U +1U)
#define ULONG_MAX (__LONG_MAX__ *2UL+1UL)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D144218.498139.patch
Type: text/x-patch
Size: 436 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20230216/b0a88edd/attachment.bin>
More information about the cfe-commits
mailing list