diff options
| author | Chris Lattner <sabre@nondot.org> | 2010-08-03 16:48:42 +0000 |
|---|---|---|
| committer | Chris Lattner <sabre@nondot.org> | 2010-08-03 16:48:42 +0000 |
| commit | 8074ab3c3cda3534e595c67f485d7711c48b493d (patch) | |
| tree | 1fe0e62995b931f62d0a3c87cb7a99dea0b66521 /lib/Frontend/InitPreprocessor.cpp | |
| parent | 2dffe2d9d631fefd79d4999f6ffe7ac0d70882b1 (diff) | |
fix some undefined behavior, PR7779.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@110116 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Frontend/InitPreprocessor.cpp')
| -rw-r--r-- | lib/Frontend/InitPreprocessor.cpp | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/lib/Frontend/InitPreprocessor.cpp b/lib/Frontend/InitPreprocessor.cpp index 9990fc42ba..e5a4169502 100644 --- a/lib/Frontend/InitPreprocessor.cpp +++ b/lib/Frontend/InitPreprocessor.cpp @@ -169,9 +169,10 @@ static void DefineTypeSize(llvm::StringRef MacroName, unsigned TypeWidth, llvm::StringRef ValSuffix, bool isSigned, MacroBuilder& Builder) { long long MaxVal; - if (isSigned) - MaxVal = (1LL << (TypeWidth - 1)) - 1; - else + if (isSigned) { + assert(TypeWidth != 1); + MaxVal = ~0ULL >> (65-TypeWidth); + } else MaxVal = ~0LL >> (64-TypeWidth); Builder.defineMacro(MacroName, llvm::Twine(MaxVal) + ValSuffix); |