[llvm-commits] CVS: llvm/lib/Target/TargetData.cpp

[Chris Lattner]

> Remove an unnecessary predicate.
> Patch by Scott Michel.

It's not clear that this is unnecessary.  Scott, does this mean that

%X = global <{int}> <{ int 1 }>

will get alignment 1?

-Chris

>
> ---
> Diffs of the changes:  (+1 -1)
>
>  TargetData.cpp |    2 +-
>  1 files changed, 1 insertion(+), 1 deletion(-)
>
>
> Index: llvm/lib/Target/TargetData.cpp
> diff -u llvm/lib/Target/TargetData.cpp:1.98 llvm/lib/Target/ 
> TargetData.cpp:1.99
> --- llvm/lib/Target/TargetData.cpp:1.98	Fri Feb 16 16:25:34 2007
> +++ llvm/lib/Target/TargetData.cpp	Fri Feb 16 16:42:40 2007
> @@ -458,7 +458,7 @@
>
>    case Type::StructTyID: {
>      // Packed structure types always have an ABI alignment of one.
> -    if (cast<StructType>(Ty)->isPacked() && abi_or_pref)
> +    if (cast<StructType>(Ty)->isPacked())
>        return 1;
>
>      // Get the layout annotation... which is lazily created on  
> demand.
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits