[PATCH] Update LLVM 3.7 for DataLayout changes

Chris Lattner via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 24 21:29:06 PDT 2015


The patch as it stands looks fine to me.

However, the follow-on question of whether it makes sense to remove/change C APIs is something the broader community should decide on going forward.

-Chris

> On Aug 24, 2015, at 9:22 PM, Mehdi Amini <mehdi.amini at apple.com> wrote:
> 
> Hi Chris,
> 
> Here is the patch.
> 
> 
> <Deprecate-the-DataLayout-on-the-TargetMachine-and-ba.patch>
> 
> Purpose is:
> 
> - Provide more information in the Release Notes about what changed with the DataLayout. This should help out-of-tree users to upgrade.
> - Deprecate LLVMGetTargetMachineData in the C API and provide hints where to look for a replacement.
> - Deprecate TargetMachine::getDataLayout(), and backport the replacement from trunk (TargetMachine::createDataLayout()), so that users don’t have to wait 3.8 to move away of a deprecated API.
> 
> Except the back-ported entry point on the TargetMachine, this is NFC.
>> Mehdi
> 
> 
> 
>> On Aug 24, 2015, at 8:59 PM, Chris Lattner <clattner at apple.com> wrote:
>> 
>> Can I get a link to the patch?
>> 
>> -Chris
>> 
>>> On Aug 24, 2015, at 5:38 PM, Hans Wennborg <hans at chromium.org> wrote:
>>> 
>>> We will need an RC4 before the final release, and this seems important
>>> enough that it could go in, but I would like to get approval from the
>>> code owner before merging.
>>> 
>>> On Fri, Aug 21, 2015 at 2:17 PM, Hans Wennborg <hans at chromium.org> wrote:
>>>> +Chris for ownership
>>>> 
>>>> On Thu, Aug 20, 2015 at 7:05 PM, Mehdi Amini <mehdi.amini at apple.com> wrote:
>>>>> Here is a patch that I’d like to get in 3.7. It:
>>>> 
>>>> Oh, why do these things always come up last-minute before the release?
>>>> Thanks for paying attention to 3.7, though :-)
>>>> 
>>>>> - provides more information in the Release notes about what changes with the DataLayout. This should help out-of-tree users.
>>>> 
>>>> Nice, thanks.
>>>> 
>>>>> - Deprecate LLVMGetTargetMachineData in the C API
>>>>> - Deprecate TargetMachine::getDataLayout(), and backports the replacement from trunk (TargetMachine::createDataLayout())
>>>> 
>>>> We've just tagged 3.7.0-rc3, and I was hoping that would become the
>>>> final release. But I can see that getting this in would be good, and
>>>> it also seems like a pretty safe change, so maybe we can squeeze it
>>>> in.
>>>> 
>>>> There's no explicit code owner for this, which means Chris owns it.
>>>> Does the patch itself look OK for you?
>>>> 
>>>> Thanks,
>>>> Hans
>> 
> 



More information about the llvm-commits mailing list