[Lldb-commits] [PATCH] D27124: [LLDB][MIPS] Fix TestWatchpointIter failure

2016-12-09 Thread Nitesh Jain via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL289211: [LLDB][MIPS] Fix TestWatchpointIter failure (authored by nitesh.jain). Changed prior to commit: https://reviews.llvm.org/D27124?vs=80414=80895#toc Repository: rL LLVM

[Lldb-commits] [PATCH] D27124: [LLDB][MIPS] Fix TestWatchpointIter failure

2016-12-06 Thread Jim Ingham via Phabricator via lldb-commits
jingham accepted this revision. jingham added a comment. This revision is now accepted and ready to land. Great, thanks! https://reviews.llvm.org/D27124 ___ lldb-commits mailing list lldb-commits@lists.llvm.org

[Lldb-commits] [PATCH] D27124: [LLDB][MIPS] Fix TestWatchpointIter failure

2016-12-06 Thread Nitesh Jain via Phabricator via lldb-commits
nitesh.jain updated this revision to Diff 80414. nitesh.jain added a comment. Update diff as per suggestion https://reviews.llvm.org/D27124 Files: source/Target/StopInfo.cpp Index: source/Target/StopInfo.cpp === ---

[Lldb-commits] [PATCH] D27124: [LLDB][MIPS] Fix TestWatchpointIter failure

2016-12-01 Thread Jim Ingham via Phabricator via lldb-commits
jingham added a comment. This looks fine. Can you add a comment explaining why this is necessary, it isn't obvious right off the bat? If this is fixing a test case, then just add a comment and this change is fine. If the fix is test-suite neutral, then please add a test case.

[Lldb-commits] [PATCH] D27124: [LLDB][MIPS] Fix TestWatchpointIter failure

2016-12-01 Thread Greg Clayton via Phabricator via lldb-commits
clayborg resigned from this revision. clayborg removed a reviewer: clayborg. clayborg added a comment. I let Jim OK this patch. https://reviews.llvm.org/D27124 ___ lldb-commits mailing list lldb-commits@lists.llvm.org

[Lldb-commits] [PATCH] D27124: [LLDB][MIPS] Fix TestWatchpointIter failure

2016-11-30 Thread Nitesh Jain via Phabricator via lldb-commits
nitesh.jain updated the summary for this revision. nitesh.jain updated this revision to Diff 79864. nitesh.jain added a comment. Updated diff as per suggestion. https://reviews.llvm.org/D27124 Files: source/Target/StopInfo.cpp Index: source/Target/StopInfo.cpp