teemperor added a comment.
Do we also want to get rid of the `// C Includes` comments that are in some
files? LLVM isn't using them either and just removing lines doesn't make git
blame more complicated.
Comment at: source/Plugins/Instruction/MIPS/EmulateInstructionMIPS.cpp:3
kubamracek added a comment.
Ping.
https://reviews.llvm.org/D44060
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
kubamracek added a reviewer: aprantl.
kubamracek added a comment.
Ping.
https://reviews.llvm.org/D44081
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
kubamracek added a reviewer: aprantl.
kubamracek added a comment.
Ping.
https://reviews.llvm.org/D44073
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
kubamracek added a reviewer: aprantl.
kubamracek added a comment.
Ping.
https://reviews.llvm.org/D43884
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
JDevlieghere added a comment.
In https://reviews.llvm.org/D54385#1294534, @teemperor wrote:
> Do we also want to get rid of the `// C Includes` comments that are in some
> files? LLVM isn't using them either and just removing lines doesn't make git
> blame more complicated.
Yup, I'll do that
kubamracek updated this revision to Diff 173603.
kubamracek added a comment.
Updating patch for the recently-landed StackFrameRecognizers.
https://reviews.llvm.org/D43886
Files:
include/lldb/API/SBThread.h
include/lldb/Target/StackFrameRecognizer.h
include/lldb/Target/Thread.h
source/AP
kubamracek added a comment.
Hi, I've updated this patch after a while. Since then, StackFrameRecognizers
have already landed, so this patch now just adds one subclass of a
StackFrameRecognizer.
> It seems however like providing some interesting ValueObjects is a pretty
> common thing to do, so