[PATCH] D32771: Verify that no compile units share the same line table in "llvm-dwarfdump --verify"

Greg Clayton via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 2 18:13:49 PDT 2017

Previous message: [PATCH] D32771: Verify that no compile units share the same line table in "llvm-dwarfdump --verify"
Next message: [PATCH] D32771: Verify that no compile units share the same line table in "llvm-dwarfdump --verify"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

clayborg added a comment.

It is fine to create a DWARFVerifier.cpp, but can we do that in the next patch? Do you really want to review a patch where everything is moved around? And all functionality is moved around? This patch is nice and self contained.


https://reviews.llvm.org/D32771

Previous message: [PATCH] D32771: Verify that no compile units share the same line table in "llvm-dwarfdump --verify"
Next message: [PATCH] D32771: Verify that no compile units share the same line table in "llvm-dwarfdump --verify"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list