https://github.com/efriedma-quic approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/66716
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
efriedma-quic wrote:
(Not sure if you have commit access; let me know if you want me to merge for
you.)
https://github.com/llvm/llvm-project/pull/66716
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/list
brendandahl wrote:
@AaronBallman or @efriedma-quic ping are you able to add reviewers?
https://github.com/llvm/llvm-project/pull/66716
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
brendandahl wrote:
@efriedma-quic missed your comment. I don't have commit access. Can you merge
for me?
Thanks!
https://github.com/llvm/llvm-project/pull/66716
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/ma
https://github.com/efriedma-quic closed
https://github.com/llvm/llvm-project/pull/66716
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/brendandahl created
https://github.com/llvm/llvm-project/pull/66716
Previously, annotations were only emitted for function definitions. With this
change annotations are also emitted for declarations. Also, emitting function
annotations is now deferred until the end so that t
llvmbot wrote:
@llvm/pr-subscribers-clang
Changes
Previously, annotations were only emitted for function definitions. With this
change annotations are also emitted for declarations. Also, emitting function
annotations is now deferred until the end so that the most up to date
declaration
brendandahl wrote:
This is relanding the patch from [here](https://reviews.llvm.org/D156172). It
fixes the [backout
failure](https://reviews.llvm.org/rG88b7e06dcf9723d0869b0c6bee030b4140e4366d)
and adds a test for it.
https://github.com/llvm/llvm-project/pull/66716
___
brendandahl wrote:
@efriedma-quic could you re-review? The only changes were
https://github.com/llvm/llvm-project/pull/66716/files#diff-e724febedab9c1a2832bf2056d208ff02ddcb2e6f90b5a653afc9b19ac78a5d7R3098-R3100
https://github.com/llvm/llvm-project/pull/66716
___
