ellis accepted this revision. ellis added a comment. This revision is now accepted and ready to land.
In D157913#4646201 <https://reviews.llvm.org/D157913#4646201>, @zequanwu wrote: > In D157913#4646184 <https://reviews.llvm.org/D157913#4646184>, @ellis wrote: > >> It looks like `debug-info-correlate-coverage.ll` was renamed twice. Is this >> intended? > > I just moved > `llvm/test/Instrumentation/InstrProfiling/debug-info-correlate-coverage.ll.` > to > `llvm/test/Instrumentation/InstrProfiling/debug-info-correlate-byte-coverage.ll` > and created a new test file > `llvm/test/Instrumentation/InstrProfiling/debug-info-correlate-clang-coverage.ll.` Got it! Sorry I got confused by the UI. Sounds good to me! I'm excited to see clang instrumentation can take full advantage of debug info correlation! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D157913/new/ https://reviews.llvm.org/D157913 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
