[PATCH] Update LLVM 3.7 for DataLayout changes

[Mehdi Amini via llvm-commits]

Hi Chris,

Here is the patch.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: Deprecate-the-DataLayout-on-the-TargetMachine-and-ba.patch
Type: application/octet-stream
Size: 2920 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150824/6b9fbe5e/attachment.obj>
-------------- next part --------------


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
>