Anyone want to take a look at this and tell me if I am on a reasonable path?<br><br><div class="gmail_quote">On Wed, Jan 6, 2010 at 12:45 PM, Talin <span dir="ltr"><<a href="mailto:viridia@gmail.com">viridia@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">This patch adds a UnionType to DerivedTypes.h. It also adds code to the bitcode reader / writer and the assembly parser for the new type, as well as a tiny .ll test file in test/Assembler. It does not contain any code related to code generation or type layout - I wanted to see if this much was acceptable before I proceeded any further.<div>


<br></div><div>Unlike my previous patch, in which the Union type was implemented as a packing option to type Struct (thereby re-using the machinery of Struct), this patch defines Union as a completely separate type from Struct.</div>


<div><br></div><div>I was a little uncertain as to how to write the tests. I'd particularly like to write tests for the bitcode reader/writer stuff, but I am not sure how to do that.</div><div><div><br>-- <br>-- Talin<br>



</div></div>
</blockquote></div><br><br clear="all"><br>-- <br>-- Talin<br>