steveire added a comment.

In D96665#2578635 <https://reviews.llvm.org/D96665#2578635>, @goncharov wrote:

> Hi @steveire! Sorry but I reverted this revert. Could you please give some 
> context of "discussed elsewhere" and "pre-existing unit test for the 
> matcher"? @aaron.ballman for cc

@njames93 didn't realize that the offending commits were from 2019, as I wrote 
in the comment. That's what I told him "elsewhere".

I plan to re-apply this revert. If you have a reason I shouldn't do that, 
please say what it is.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96665

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

Reply via email to