Fix two classes of bugs. First:
authorChris Lattner <sabre@nondot.org>
Mon, 9 Mar 2009 04:49:14 +0000 (04:49 +0000)
committerChris Lattner <sabre@nondot.org>
Mon, 9 Mar 2009 04:49:14 +0000 (04:49 +0000)
commita9a9e07d1c5d3c73835e716d81c2ec94ad0b865f
treed4dae89d38da9d4f6e47fc1583a44fbd4d9bec3e
parentc66996ab3451a29a207e7871b0cbf984beaf941c
Fix two classes of bugs.  First:

validate an invariant so that the asmparser rejects a bad construct
instead of the verifier.  Before:

llvm-as: assembly parsed, but does not verify as correct!
Invalid struct return type!
i64 (%struct.Type*, %struct.Type*)* @foo

after:

llvm-as: t.ll:5:8: functions with 'sret' argument must return void
define i64 @foo(%struct.Type* noalias nocapture sret %agg.result, %struct.Type* nocapture byval %t) nounwind {
       ^

Second, check that void is only used where allowed (in function return types) not in
arbitrary places, fixing PR3747 - Crash in llvm-as with void field in struct.  We
now reject that example with:

$ llvm-as t.ll
llvm-as: t.ll:1:12: struct element can not have void type
%x = type {void}
           ^

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@66394 91177308-0d34-0410-b5e6-96231b3b80d8
lib/AsmParser/LLParser.cpp
lib/AsmParser/LLParser.h