This tool's job is to dump the vtables inside object files. It is
currently limited to MS ABI vf- and vb-tables but it will eventually
support Itanium-style v-tables as well.
Details
Diff Detail
Event Timeline
tools/llvm-vtabledump/Error.cpp | ||
---|---|---|
20 | Why the leading underscore? | |
27 | How about just defining this and message() in the class? | |
tools/llvm-vtabledump/Error.h | ||
36 | Is this OK because we're not adding anything to namespace std, just specializing an already existing template in there? | |
tools/llvm-vtabledump/llvm-vtabledump.cpp | ||
58 | Could this just call reportError(Input, EC.message()) ? | |
71 | Should this flush like the other reportError does? | |
82 | Maybe a comment about what ??_7 signifies? Maybe the stuff in this if branch can be broken out to a helper function? (These comments apply to the ??_8 side below too.) | |
135 | IIRC, \brief isn't necessary on one-line doxygen comments. Applies below too. | |
tools/llvm-vtabledump/llvm-vtabledump.h | ||
1 | I thought we usually put "C++" in these things, but not in the tools/ dir? | |
10 | Wrong include guard name. |
Is this OK because we're not adding anything to namespace std, just specializing an already existing template in there?