<div dir="ltr"><div class="gmail_quote"><div>(Explicitly adding a few more people for visibility.)</div><div dir="ltr"><br></div><div dir="ltr">On Mon, 6 Aug 2018 at 13:19, Stephen Kelly via cfe-dev <<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Stephen Kelly via cfe-dev wrote:<br>
<br>
> Richard Smith via cfe-dev wrote:<br>
>> We have historically taken the position that the churn caused for<br>
>> out-of-tree users of the C++ API are not a consideration when performing<br>
>> an API refactoring. (For two reasons: one is that we explicitly do not<br>
>> have a stable C++ API; the other is that this is the mechanism by which<br>
>> we encourage people to upstream changes.) This actually seems like a<br>
>> relatively easy change in that regard: client code can be largely updated<br>
>> by a regular expression.<br>
> <br>
> Great, I'll look into doing this in a bit.<br>
<br>
I've pushed some Phab reviews to complete the port.  For convenience, the <br>
commits can be seen together on my github:<br>
<br>
 <a href="https://github.com/steveire/clang/commits/API-cleanup" rel="noreferrer" target="_blank">https://github.com/steveire/clang/commits/API-cleanup</a><br>
<br>
 <a href="https://github.com/steveire/clang-tools-extra/commits/API-cleanup" rel="noreferrer" target="_blank">https://github.com/steveire/clang-tools-extra/commits/API-cleanup</a><br>
<br>
My intention is to push all except the actual removal of the old API at <br>
first, then push the removal of the old API 3 or so weeks later.<br></blockquote><div><br></div><div>Thanks, the patches all look fine to me, but please use the LLVM_ATTRIBUTE_DEPRECATED macro rather than [[deprecated]] for the deprecation patch.</div><div><br></div><div>Please also ping <a href="https://reviews.llvm.org/D50353" rel="noreferrer" target="_blank">https://reviews.llvm.org/D50353</a> when you think it's time to commit that; we should make sure that people tracking SVN trunk have had enough time to move to the new names before removing the old ones.</div><div><br></div><div>Also, please make sure the LLVM subprojects that use clang as a library (lldb, clang-tools-extra) are updated.</div></div></div>