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
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
tools/llvm-vtabledump/Error.cpp | ||
---|---|---|
20 ↗ | (On Diff #11653) | Why the leading underscore? |
27 ↗ | (On Diff #11653) | How about just defining this and message() in the class? |
tools/llvm-vtabledump/Error.h | ||
36 ↗ | (On Diff #11653) | 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 ↗ | (On Diff #11653) | Could this just call reportError(Input, EC.message()) ? |
71 ↗ | (On Diff #11653) | Should this flush like the other reportError does? |
82 ↗ | (On Diff #11653) | 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 ↗ | (On Diff #11653) | IIRC, \brief isn't necessary on one-line doxygen comments. Applies below too. |
tools/llvm-vtabledump/llvm-vtabledump.h | ||
1 ↗ | (On Diff #11653) | I thought we usually put "C++" in these things, but not in the tools/ dir? |
10 ↗ | (On Diff #11653) | Wrong include guard name. |