[patch] Simplify the datalayout strings by using the defaults

Rafael EspĂ­ndola rafael.espindola at gmail.com
Wed Dec 11 11:36:21 PST 2013


On 11 December 2013 14:14, Tim Northover <t.p.northover at gmail.com> wrote:
> On 11 December 2013 17:45, Rafael EspĂ­ndola <rafael.espindola at gmail.com> wrote:
>> The datalayout string has some useful defaults, but the targets are
>> fairly inconsistent about which defaults they use.
>
> If you're overhauling the targets anyway, "i64:32:64" stands out as a
> very odd default.

Yes, I was not sure about changing defaults on the same commit. It
might require updating test that use opt. We should probably also add
a i128 default.

In any case, would you be OK with the default changing being an
independent commit?

Cheers,
Rafael




More information about the llvm-commits mailing list