[PATCH] D21982: WIP: Implement a per-thread inmemory log

[Dean Michael Berris via llvm-commits]

dberris added a comment.

Question for Reid/Medhi/Eric:

Now that https://reviews.llvm.org/D21612 has landed I'm thinking of adding tests here to be able to do some sanity-checking of the generated log file. That though means that the "best" means of doing this is by using a tool that will output the information in a verifiable manner. We have some options in that regard:

- Implement a stand-alone tool in compiler-rt just for testing. This seems brittle but should be simple enough to do.
- Land https://reviews.llvm.org/D21987 in LLVM and then potentially depend on that from compiler-rt (although I'm not sure if the dependency chain would be correct).

I'm leaning towards the first solution, but I'd like to hear your thoughts on how best to go about this too.

Cheers


https://reviews.llvm.org/D21982