Skip to content

[lldb] Remove stack_memory from MemoryRegionInfo constructor - #189580

Merged
DavidSpickett merged 1 commit into
llvm:mainfrom
DavidSpickett:lldb-mem2
Mar 31, 2026
Merged

[lldb] Remove stack_memory from MemoryRegionInfo constructor#189580
DavidSpickett merged 1 commit into
llvm:mainfrom
DavidSpickett:lldb-mem2

Conversation

@DavidSpickett

Copy link
Copy Markdown
Contributor

Nothing passes anything but the default value here. Code that wants to set it is starting with a default constructed info, and filling in the fields one by one (the gdb-remote client for example).

Nothing passes anything but the default value here. Code that
wants to set it is starting with a default constructed info,
and filling in the fields one by one (the gdb-remote client
for example).
@DavidSpickett DavidSpickett added the skip-precommit-approval PR for CI feedback, not intended for review label Mar 31, 2026
@DavidSpickett
DavidSpickett enabled auto-merge (squash) March 31, 2026 10:10
@llvmbot llvmbot added the lldb label Mar 31, 2026
@DavidSpickett
DavidSpickett merged commit c6b255a into llvm:main Mar 31, 2026
12 of 13 checks passed
@DavidSpickett
DavidSpickett deleted the lldb-mem2 branch March 31, 2026 11:29
zwu-2025 pushed a commit to zwu-2025/llvm-project that referenced this pull request May 17, 2026
…9580)

Nothing passes anything but the default value here. Code that wants to
set it is starting with a default constructed info, and filling in the
fields one by one (the gdb-remote client for example).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lldb skip-precommit-approval PR for CI feedback, not intended for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants