[Lldb-commits] [PATCH] D73148: [lldb/Value] Avoid reading more data than the host has available
Vedant Kumar via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Thu Jan 30 15:26:17 PST 2020
vsk marked an inline comment as done.
vsk added inline comments.
================
Comment at: lldb/packages/Python/lldbsuite/test/functionalities/dw_op_piece/main.cpp:34
+ //% self.filecheck("image lookup -va $pc", "main.cpp", "-check-prefix=INFO-V1")
+ // INFO-V1: name = "v1", type = "S1", location = DW_OP_reg0 RAX, DW_OP_piece 0x4, DW_OP_piece 0x4, DW_OP_lit0, DW_OP_stack_value, DW_OP_piece 0x8, decl
+ sink++;
----------------
aprantl wrote:
> It seems unlikely that this lowering survives very long. Could you move this to unit tests instead?
I'm not sure how to write this as a unit test. I expect that the added regression test + assembly test covers what we want to cover, though.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D73148/new/
https://reviews.llvm.org/D73148
More information about the lldb-commits
mailing list