mgorny created this revision.
mgorny added reviewers: labath, zturner, krytarowski.
Do not use -nostdlib in target-symbols-add-unwind.test. NetBSD uses
startup files to provide obligatory ELF notes in executables,
and therefore using -nostdlib requires providing specially tailored
input. Otherwi
Author: mgorny
Date: Sat Apr 13 02:49:39 2019
New Revision: 358329
URL: http://llvm.org/viewvc/llvm-project?rev=358329&view=rev
Log:
[lldb] [lit/SymbolFile] Avoid -nostdlib in target-symbols-add-unwind.test
Do not use -nostdlib in target-symbols-add-unwind.test. NetBSD uses
startup files to prov
This revision was automatically updated to reflect the committed changes.
Closed by commit rGfb70bc65fe18: [lldb] [lit/SymbolFile] Avoid -nostdlib in
target-symbols-add-unwind.test (authored by mgorny).
Herald added a project: LLDB.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
teemperor created this revision.
teemperor added a reviewer: clayborg.
Herald added a project: LLDB.
Herald added a subscriber: lldb-commits.
Fixes LLVM bug 41489.
Repository:
rLLDB LLDB
https://reviews.llvm.org/D60653
Files:
lldb/source/Core/Module.cpp
Index: lldb/source/Core/Module.cpp
teemperor created this revision.
teemperor added a reviewer: LLDB.
Herald added subscribers: lldb-commits, kristof.beyls, javed.absar.
Herald added a project: LLDB.
As reported in LLVM bug 41487, the check in this function is wrong and should be
the same as the described check in the comment (whic
teemperor created this revision.
teemperor added a reviewer: LLDB.
Herald added subscribers: lldb-commits, kristof.beyls, javed.absar.
Herald added a project: LLDB.
As reported in LLVM bug 41486, the check `(byte1 & 0xf8) == 0xc0` is wrong. We
want to check for `11010nnn`,
so the proper value we
teemperor added a comment.
Don't have the test for this and probably can't write one for it (as I don't
have access to an ARM device for testing and don't really know the code so
well). If anyone feels like providing a test, feel free to link it to this
revision.
Repository:
rLLDB LLDB
CHA
davide accepted this revision.
davide added a comment.
This revision is now accepted and ready to land.
Add an unittest maybe? otherwise, seems to match the ISA so LGTM
Repository:
rLLDB LLDB
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D60654/new/
https://reviews.llvm.org/D60654