[Lldb-commits] [PATCH] D38897: Add specific ppc64le hardware watchpoint handler

[Zachary Turner via Phabricator via lldb-commits]

zturner added a comment.

In https://reviews.llvm.org/D38897#897378, @gut wrote:

> Thanks for supporting this change. I guess @anajuliapc will add you both as reviewer as soon as she updates this patch.
>
> BTW, I agree that patches should be improving code quality but I wanted to highlight that these changes were actually based on the current ARM64 implementation. E.g: this snip <https://github.com/llvm-mirror/lldb/blob/master/source/Plugins/Process/Linux/NativeRegisterContextLinux_arm64.cpp#L609> and this requested change <https://reviews.llvm.org/D38897#inline-339476>. But thanks for pointing it out.


I'd be equally fine with extracting that logic into a separate function, it looks mostly the same?  I don't think we should be duplicating code (or even algorithms written slightly differently)


https://reviews.llvm.org/D38897