[PATCH] D36250: [coverage] Special-case calls to noreturn functions.

Vedant Kumar via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 4 17:50:19 PDT 2017


vsk added a comment.

I tested this out on clang's ItaniumMangle.cpp and the crash is resolved.



================
Comment at: lib/CodeGen/CoverageMappingGen.cpp:723
+      this->Visit(Child);
+    handleFileExit(getEnd(E));
+
----------------
This is doing the right thing. I think it should just be replaced by a call to VisitStmt(E), though, as that's clearer.


Repository:
  rL LLVM

https://reviews.llvm.org/D36250





More information about the cfe-commits mailing list