User Details
- User Since
- Aug 14 2017, 4:02 AM (179 w, 5 h)
Oct 27 2020
Uploaded a patch that corrects a merge conflict.
Oct 21 2020
@MaskRay: We have added support for Split DWARF. We would appreciate, if you can try again llvm-dva with the binaries you used in your test cases. Thanks.
Following suggestions from @echristo in relation with the tool name, it was decided to renamed it as llvm-dva.
Oct 8 2020
Uploaded an updated patch:
Oct 4 2020
Uploaded a correct patch that includes:
% fllvm-diva --print=scopes /tmp/Debug/bin/clang Logical View: {File} '/tmp/debug/bin/clang' {CompileUnit} 'cc1_main.cpp.dwo' {CompileUnit} 'cc1as_main.cpp.dwo' {CompileUnit} 'cc1gen_reproducer_main.cpp.dwo' {CompileUnit} 'driver.cpp.dwo'Are 'instructions', 'symbols', and 'types' unavailable?
% fllvm-diva --print=symbols /tmp/Debug/bin/clang Logical View: % fllvm-diva --print=types /tmp/Debug/bin/clang Logical View:
Some documented options appear to be unavailable.
% fllvm-diva --sort=offset /tmp/Debug/bin/clang llvm-diva: Unknown command line argument '--sort=offset'. Try: '/tmp/RelA/bin/llvm-diva --help' llvm-diva: Did you mean '--report=offset'?
For CMake changes, it'd be good to test a -DBUILD_SHARED_LIBS=on configuration. Many unspecified dependency can be caught. It also pushes the author to think whether certain dependencies are needed.
FAILED: lib/libLLVMDebugInfoLogicalView.so.12git ... ld.lld: error: undefined symbol: llvm::raw_ostream::write(char const*, unsigned long) ld.lld: error: undefined symbol: llvm::Twine::str[abi:cxx11]() const
@MaskRay Thanks very much for your valuable comments.
Oct 2 2020
This patch includes the a/ b/ prefixes.
Oct 1 2020
Apr 23 2020
Apr 4 2020
Apr 3 2020
Feb 11 2019
@aprantl, as per your feedback I have moved the test case to the correct location.
Feb 8 2019
Feb 7 2019
Updated patch to address comments from @aprantl .
Jan 31 2019
Updated patch to remove the negative CHECK. Thanks for your feedback.
Thanks very much for your invaluable feedback and reviews.
Jan 30 2019
Jan 15 2019
Jan 10 2019
Nov 13 2018
Your change seems to cause build failures. The following is the error message:
Nov 8 2018
@vsk: I have updated the patch to remove the test case pr39187-gline-tables-only.ll, as per your suggestion.
Nov 7 2018
Thanks for your comments.
Nov 6 2018
Nov 5 2018
Oct 30 2018
Thanks for your comments. I have updated the patch to address them.
Oct 29 2018
Ping.
Oct 25 2018
@courbet: Your commit seems to cause build issues:
Oct 24 2018
@vsk: Do you have any additional comments? It is OK for you if I commit the patch?
Oct 18 2018
Oct 16 2018
Address issues raised by @aprantl.
Oct 15 2018
This revision caused builbot failures.
Oct 11 2018
The patch caused some assertions. For more details see PR39243.
Oct 10 2018
@mstorsjo, The change has been reverted. Thanks for creating the bugzilla.
Thanks very much to all the reviewers for your valuable comments and suggestions.
Oct 9 2018
Address issues raised by the reviewers.
Oct 8 2018
Address issues raised by the reviewers.
Oct 5 2018
Address issues raised by the reviewers.
Oct 4 2018
Oct 1 2018
Thanks very much to all the reviewers for your valuable comments and suggestions.
Sep 28 2018
Address issues raised by the reviewers.
Sep 27 2018
Sep 19 2018
Thanks very much to all the reviewers for your valuable comments and suggestions.
Sep 18 2018
The general preference seems to land this patch as-is.
Sep 17 2018
@aprantl and @probinson,
Address issues raised by the reviewers.
Sep 14 2018
Address issues raised by the reviewers.
Address issues raised by the reviewers.
Sep 13 2018
Sep 12 2018
Sep 3 2018
Aug 30 2018
Aug 29 2018
Aug 28 2018
Update the original patch, as the results from the investigation to replace the algorithm used by 'collectDebugValues', showed some additional complexities.
Aug 24 2018
Aug 21 2018
Following feedback from the reviewers:
Aug 20 2018
Aug 17 2018
Jul 26 2018
Is there anything I can add to this patch in order to submit it?
This patch is blocking the review https://reviews.llvm.org/D44826 which is already approved.
Jul 20 2018
A minor modification to allow the comments for MarkNamespaceAliasReferenced to be picked up by doxygen.