[llvm-commits] [PATCH] Make opt default to not adding a target data string, and update tests

[Dan Gohman]

Hello,

If I read this correctly, this makes the NoDefaultDataLayout
option unused. Is that right? If so, please remove the option
completely.

Otherwise, this patch looks fine. Thanks for updating all those tests!

Dan

On Nov 2, 2009, at 11:13 AM, Kenneth Uildriks wrote:

> All tests in the test directory that depend on target data now
> explicitly include the target data in the input assembly.  This makes
> it safe for opt to not add target data whenever it's not supplied by
> either the module or the command line, which in turn stops opt from
> breaking valid code with bogus target data assumptions.
> <tests-opt.patch>_______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits