diff options
| author | Georgii Rymar <grimar@accesssoftek.com> | 2020-08-04 15:19:36 +0300 |
|---|---|---|
| committer | Georgii Rymar <grimar@accesssoftek.com> | 2020-08-05 13:09:04 +0300 |
| commit | f97019ad6e3a96995dda3f759ee692eb81abcc4c (patch) | |
| tree | 05b3d3fd20e52899bf92158210e97a99b38df56e /llvm/lib/CodeGen/StackMaps.cpp | |
| parent | c558c22cab9a555d2e521102b775759381e9727f (diff) | |
| download | llvm-f97019ad6e3a96995dda3f759ee692eb81abcc4c.zip llvm-f97019ad6e3a96995dda3f759ee692eb81abcc4c.tar.gz llvm-f97019ad6e3a96995dda3f759ee692eb81abcc4c.tar.bz2 | |
[llvm-readobj/elf] - Add a testing for --stackmap and refine the implementation.
Currently, we only test the `--stackmap` option here:
https://github.com/llvm/llvm-project/blob/master/llvm/test/Object/stackmap-dump.test
it uses a precompiled MachO binary currently and I've found no tests for this option for ELF.
The implementation also has issues. For example, it might assert on a wrong version
of the .llvm-stackmaps section. Or it might crash on an empty or truncated section.
This patch introduces a new tools/llvm-readobj/ELF test file as well as implements a few
basic checks to catch simple crashes/issues
It also eliminates `unwrapOrError` calls in `printStackMap()`.
Differential revision: https://reviews.llvm.org/D85208
Diffstat (limited to 'llvm/lib/CodeGen/StackMaps.cpp')
| -rw-r--r-- | llvm/lib/CodeGen/StackMaps.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/CodeGen/StackMaps.cpp b/llvm/lib/CodeGen/StackMaps.cpp index 1e060ec..113d477 100644 --- a/llvm/lib/CodeGen/StackMaps.cpp +++ b/llvm/lib/CodeGen/StackMaps.cpp @@ -404,7 +404,7 @@ void StackMaps::recordStatepoint(const MCSymbol &L, const MachineInstr &MI) { /// Emit the stackmap header. /// /// Header { -/// uint8 : Stack Map Version (currently 2) +/// uint8 : Stack Map Version (currently 3) /// uint8 : Reserved (expected to be 0) /// uint16 : Reserved (expected to be 0) /// } |