[PATCH] D109470: Add "profiling" to the list of absl libraries.

2021-09-09 Thread Yitzhak Mandelbaum via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGe976fc61ecd9: Add profiling to the list of absl libraries. (authored by nilayvaish, committed by ymandel). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D109470: Add "profiling" to the list of absl libraries.

2021-09-09 Thread Yitzhak Mandelbaum via Phabricator via cfe-commits
ymandel added a comment. In D109470#2992275 , @nilayvaish wrote: > ymandel@, I do not have commit access to the repo. Can you commit this for > me? Thanks! Sure! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D109470: Add "profiling" to the list of absl libraries.

2021-09-09 Thread Nilay Vaish via Phabricator via cfe-commits
nilayvaish added a comment. ymandel@, I do not have commit access to the repo. Can you commit this for me? Thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109470/new/ https://reviews.llvm.org/D109470

[PATCH] D109470: Add "profiling" to the list of absl libraries.

2021-09-09 Thread Yitzhak Mandelbaum via Phabricator via cfe-commits
ymandel added a comment. In D109470#2992161 , @nilayvaish wrote: > In D109470#2992119 , @ymandel wrote: > >> Any idea if there's a test for this matcher that should be updated? > > I looked at the commit that

[PATCH] D109470: Add "profiling" to the list of absl libraries.

2021-09-09 Thread Nilay Vaish via Phabricator via cfe-commits
nilayvaish added a comment. In D109470#2992119 , @ymandel wrote: > Any idea if there's a test for this matcher that should be updated? I looked at the commit that most recently added a directory to this file: 114c9fa0e46f7bf1d05d92da70da116b19f16911

[PATCH] D109470: Add "profiling" to the list of absl libraries.

2021-09-09 Thread Yitzhak Mandelbaum via Phabricator via cfe-commits
ymandel accepted this revision. ymandel added a comment. This revision is now accepted and ready to land. Any idea if there's a test for this matcher that should be updated? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109470/new/

[PATCH] D109470: Add "profiling" to the list of absl libraries.

2021-09-08 Thread Nilay Vaish via Phabricator via cfe-commits
nilayvaish created this revision. nilayvaish requested review of this revision. Herald added a project: clang-tools-extra. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D109470 Files: clang-tools-extra/clang-tidy/abseil/AbseilMatcher.h