From: Sanjiv Gupta <sanjiv.gupta@microchip.com>
Date: Tue, 14 Apr 2009 16:46:37 +0000 (+0000)
Subject: Literal value calculation isn't likely to overflow on targets having int as 32 or... 
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=0fab49f7daa53a53882f19c2879feb210608fe00;p=clang

Literal value calculation isn't likely to overflow on targets having int as 32 or less. Fixing the assert as it otherwise triggers for PIC16 which as i16 as int.


git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@69046 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/Lex/LiteralSupport.cpp b/lib/Lex/LiteralSupport.cpp
index a52d951e75..e8d3162886 100644
--- a/lib/Lex/LiteralSupport.cpp
+++ b/lib/Lex/LiteralSupport.cpp
@@ -632,10 +632,10 @@ CharLiteralParser::CharLiteralParser(const char *begin, const char *end,
   assert(begin[0] == '\'' && "Invalid token lexed");
   ++begin;
 
-  // FIXME: This assumes that 'int' is 32-bits in overflow calculation, and the
-  // size of "value".
-  assert(PP.getTargetInfo().getIntWidth() == 32 &&
-         "Assumes sizeof(int) == 4 for now");
+  // FIXME: This assumes that 'int' is not more than 32-bits in overflow 
+  // calculation, and the size of "value".
+  assert(PP.getTargetInfo().getIntWidth() <= 32 &&
+         "Assumes sizeof(int) <= 4 for now");
   // FIXME: This assumes that wchar_t is 32-bits for now.
   assert(PP.getTargetInfo().getWCharWidth() == 32 && 
          "Assumes sizeof(wchar_t) == 4 for now");