[PATCH] D12538: Fix assertion failure in LLParser::ConvertValIDToValue

[Karl Schimpf via llvm-commits]

kschimpf created this revision.
kschimpf added a reviewer: kcc.
kschimpf added a subscriber: llvm-commits.

Fixes bug 24645. Problem appears to be that the type may be undefined
when ConvertValIDToValue is called.

http://reviews.llvm.org/D12538

Files:
  lib/AsmParser/LLParser.cpp
  lib/AsmParser/LLParser.h

Index: lib/AsmParser/LLParser.h
===================================================================
--- lib/AsmParser/LLParser.h
+++ lib/AsmParser/LLParser.h
@@ -59,7 +59,7 @@
 
     LLLexer::LocTy Loc;
     unsigned UIntVal;
-    FunctionType *FTy;
+    FunctionType *FTy = nullptr;
     std::string StrVal, StrVal2;
     APSInt APSIntVal;
     APFloat APFloatVal{0.0};
Index: lib/AsmParser/LLParser.cpp
===================================================================
--- lib/AsmParser/LLParser.cpp
+++ lib/AsmParser/LLParser.cpp
@@ -4128,8 +4128,7 @@
     V = PFS->GetVal(ID.StrVal, Ty, ID.Loc, OC);
     return V == nullptr;
   case ValID::t_InlineAsm: {
-    assert(ID.FTy);
-    if (!InlineAsm::Verify(ID.FTy, ID.StrVal2))
+    if (!ID.FTy || !InlineAsm::Verify(ID.FTy, ID.StrVal2))
       return Error(ID.Loc, "invalid type for inline asm constraint string");
     V = InlineAsm::get(ID.FTy, ID.StrVal, ID.StrVal2, ID.UIntVal & 1,
                        (ID.UIntVal >> 1) & 1,


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D12538.33722.patch
Type: text/x-patch
Size: 988 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150901/328e471c/attachment.bin>