[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