[PATCH] D14579: sancov tool in c++

Mike Aizatsky via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 11 13:01:33 PST 2015


aizatsky added a comment.

> Make sure "ninja sancov" works as expected, and "ninja all" builds the tool as well.


ack.

> Also, I presume you did run clang-format over the entire source.


yes

> You might need to add Makefile support for autoconf build (just cargo-cult from another tools) - true, it's being deprecated at the moment, but we might need to keep building sancov until 3.8 release - in order to run the test-suite.


done


Repository:
  rL LLVM

http://reviews.llvm.org/D14579





More information about the llvm-commits mailing list