[cfe-dev] API Removal Notice (4 weeks): getStartLoc, getLocStart, getLocEnd

[Stephen Kelly via cfe-dev]

Hi David,

I think that's a release manager decision (cc Hans)

The API doesn't exist in 7.0.0. All of my commits (introduce the new API,
port to it, deprecate the old API) would have to be backported.

In discussions so far, it was not a priority to keep the old API or notify
particularly sensitively.

If the totality of the change is deemed too large to backported, it could
make sense to only backport the new API and made a message to the release
notes to encourage downstreams to port.

Thanks,

Stephen.

On Fri, 10 Aug 2018, 14:44 David Chisnall, <David.Chisnall at cl.cam.ac.uk>
wrote:

> On 10 Aug 2018, at 00:13, Stephen Kelly via cfe-dev <
> cfe-dev at lists.llvm.org> wrote:
> >
> > All third-party code must be similarly updated.
> >
> > Currently, the old names remain in the code, but annotated as
> deprecated.
>
> If you haven’t already, please will you add this change to the set to be
> backported to the 7.0 branch?  The deprecation notice was really useful -
> it told me exactly what I needed to change to make it go away - but if it
> never ships in a major release then a lot of downstream users will never
> see it.
>
> David
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20180810/bf7f8247/attachment.html>