[PATCH] D25434: Generate more debug info in -gmlt

Dehao Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 6 10:51:47 PST 2016


danielcdh added a comment.

In https://reviews.llvm.org/D25434#614157, @echristo wrote:

> With Minimal and the ProfileDebug it seems that the function can be rearranged to make the logic a bit more clear. Also needs comments describing what we're looking for as far as profile debug. (The clang flag also needs documentation come to think of it.)


Added some comment in the new version. For the clang flag documentation, do you mean the changes in include/clang/Driver/Options.td ? Or could you point to which file I should update to change the documentation? Thanks

> Can you look into Minimal plus profile debugging and see what we can unify here? Also coverage analysis I imagine.

I'm not quite sure if I understand correctly. My understanding was that Minimal and ProfileDebug are orthogonal?

Thanks,
Dehao

> Thanks!
> 
> -eric




https://reviews.llvm.org/D25434





More information about the llvm-commits mailing list