[Lldb-commits] [PATCH] D67911: [LLDB] [Windows] Add missing ifdefs to fix building for non-x86 architectures
Martin Storsjö via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Mon Sep 23 12:08:09 PDT 2019
mstorsjo marked an inline comment as done.
mstorsjo added a comment.
In D67911#1679333 <https://reviews.llvm.org/D67911#1679333>, @compnerd wrote:
> What do you think of adding some sort of notification that hardware breakpoints are currently unsupported and that we are falling back to software breakpoints for the `#else` case?
I guess it could make sense. What kind of notification do you have in mind?
Comment at: lldb/source/Plugins/Process/Windows/Common/RegisterContextWindows.cpp:87
#if defined(__x86_64__) || defined(_M_AMD64)
> Should this line not also include `|| defined(__aarch64__) || defined(_M_ARM64)`?
I guess it should. Or this might be one of the few places where checking for just `_WIN64` might be easiest?
CHANGES SINCE LAST ACTION
More information about the lldb-commits