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?