Hi Evan, thanks for doing this.

> +static bool isZeroSizedStructOrUnion(tree type) {
> +  if (TREE_CODE(type) != RECORD_TYPE && TREE_CODE(type) != UNION_TYPE)

You should let QUAL_UNION through here as well.  And what about size-zero
arrays?  Or for that matter, zero size anything?

> +        // Skip 'int:0', which just affects layout.
> +        unsigned FieldSizeInBits = TREE_INT_CST_LOW(DECL_SIZE(Field));
> +        if (FieldSizeInBits == 0)
> +          continue;

This is probably safe (not completely clear), but why do you need to do it at 
all?

Ciao,

Duncan.
_______________________________________________
llvm-commits mailing list
llvm-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

Reply via email to