vsk added a comment.

I haven't taken a close look at the tests yet, but plan on getting to it soon.

I'm not sure whether this is something you've already tried, but for this kind 
of change, it can be helpful to verify that a stage2 clang self-host with 
assertions enabled completes successfully. For coverage specifically, it's 
possible to do that by setting '-DLLVM_BUILD_INSTRUMENTED_COVERAGE=On' during 
the stage2 cmake step.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84467

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

Reply via email to