aaron.ballman added a comment.

In D139197#3978761 <https://reviews.llvm.org/D139197#3978761>, @carlosgalvezp 
wrote:

> In D139197#3978488 <https://reviews.llvm.org/D139197#3978488>, @aaron.ballman 
> wrote:
>
>> LGTM, though please add a release note describing the change
>
> The original check is just a few days old. It didn't exist in the previous 
> release. Should I still add a comment about this change?

Oh, I hadn't realized the original check was part of this release. No need for 
a release note in this case.

In D139197#3978771 <https://reviews.llvm.org/D139197#3978771>, @carlosgalvezp 
wrote:

> PS thanks a lot for the review! :)

Happy to help where I can!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139197

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

Reply via email to