diff options
author | Chris Lattner <sabre@nondot.org> | 2009-03-09 04:49:14 +0000 |
---|---|---|
committer | Chris Lattner <sabre@nondot.org> | 2009-03-09 04:49:14 +0000 |
commit | a9a9e07d1c5d3c73835e716d81c2ec94ad0b865f (patch) | |
tree | d4dae89d38da9d4f6e47fc1583a44fbd4d9bec3e /lib/CodeGen/SelectionDAG/SelectionDAGPrinter.cpp | |
parent | c66996ab3451a29a207e7871b0cbf984beaf941c (diff) |
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
Diffstat (limited to 'lib/CodeGen/SelectionDAG/SelectionDAGPrinter.cpp')
0 files changed, 0 insertions, 0 deletions