[PATCH] Update LLVM 3.7 for DataLayout changes

Mehdi Amini via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 24 21:30:42 PDT 2015


Yes, I don’t make any assumption on the result of the current discussion with the community on this aspect, and by default I take care not to remove *anything* in the meantime :)

Thanks,

Mehdi


> On Aug 24, 2015, at 9:29 PM, Chris Lattner <clattner at apple.com> wrote:
> 
> 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