From 0247b81ef6a48412d0fd00026f26634f30efa26c Mon Sep 17 00:00:00 2001 From: David Majnemer Date: Tue, 9 Dec 2014 19:10:03 +0000 Subject: [PATCH] AsmParser: Don't crash on short hex constants for fp128 types If we see 0xL01, treat it like 0xL00000000000000000000000000000001 instead of crashing. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@223811 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/AsmParser/LLLexer.cpp | 12 +++++++----- test/Assembler/short-hexpair.ll | 4 ++++ 2 files changed, 11 insertions(+), 5 deletions(-) create mode 100644 test/Assembler/short-hexpair.ll diff --git a/lib/AsmParser/LLLexer.cpp b/lib/AsmParser/LLLexer.cpp index 614a283241e..d2ed616e782 100644 --- a/lib/AsmParser/LLLexer.cpp +++ b/lib/AsmParser/LLLexer.cpp @@ -78,13 +78,15 @@ uint64_t LLLexer::HexIntToVal(const char *Buffer, const char *End) { void LLLexer::HexToIntPair(const char *Buffer, const char *End, uint64_t Pair[2]) { Pair[0] = 0; - for (int i=0; i<16; i++, Buffer++) { - assert(Buffer != End); - Pair[0] *= 16; - Pair[0] += hexDigitValue(*Buffer); + if (End - Buffer >= 16) { + for (int i = 0; i < 16; i++, Buffer++) { + assert(Buffer != End); + Pair[0] *= 16; + Pair[0] += hexDigitValue(*Buffer); + } } Pair[1] = 0; - for (int i=0; i<16 && Buffer != End; i++, Buffer++) { + for (int i = 0; i < 16 && Buffer != End; i++, Buffer++) { Pair[1] *= 16; Pair[1] += hexDigitValue(*Buffer); } diff --git a/test/Assembler/short-hexpair.ll b/test/Assembler/short-hexpair.ll new file mode 100644 index 00000000000..067ea30b0dd --- /dev/null +++ b/test/Assembler/short-hexpair.ll @@ -0,0 +1,4 @@ +; RUN: llvm-as < %s | llvm-dis | FileCheck %s + +@x = global fp128 0xL01 +; CHECK: @x = global fp128 0xL00000000000000000000000000000001