zequanwu added a comment. In D83592#2147881 <https://reviews.llvm.org/D83592#2147881>, @hans wrote:
> > Not sure if this is good idea to untrack comments, it breaks many tests > > under CoverageMapping. > > I didn't look, but I'm surprised there are many coverage tests that have > comments in them. If the comments are not important for those tests, maybe > they could be removed? As long as we keep one that tracks the intended > behavior of coverage on comments. The comments in those coverage tests are used for FileCheck, like `//CHECK::`. So, we can't remove those ones. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D83592/new/ https://reviews.llvm.org/D83592 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
