aaron.ballman accepted this revision.
aaron.ballman added a comment.

In D121245#3382887 <https://reviews.llvm.org/D121245#3382887>, @ArcsinX wrote:

> In D121245#3380346 <https://reviews.llvm.org/D121245#3380346>, @aaron.ballman 
> wrote:
>
>> LGTM! Might be worth adding a release note for it (does this close any bugs 
>> in the bug database? If so, it'd be worth mentioning those in the commit 
>> message and release note).
>
> Thank you for review.
>
> I failed to find any opened bugs related to this. I'm not sure how detailed 
> information about this patch should be in release notes. Can you please take 
> a look at `ReleaseNotes.rst` modifications?

The release notes look great to me, thank you for adding them!


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D121245/new/

https://reviews.llvm.org/D121245

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to