IamYJLee wrote:

> LGTM, but please also add release note to 
> `clang-tools-extra/ReleaseNotes.rst` so that people know about the fix:) 
> Forgot to mention it before.. sorry. All user-visible changes should go with 
> release notes I'm not sure if we need a release note on clang side (not 
> familiar how things are done)

Oh, I forgot to add a release note.
Thanks for your feedback!

@ChuanqiXu9 Could you advise me on how to write the ReleaseNote on the Clang 
side?

https://github.com/llvm/llvm-project/pull/201481
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to