mohit.bhakkad added a comment.

I forgot about note part, is it okay to add a TODO before ignore count 
condition in source/Target/StopInfo.cpp:

TODO: This condition should be checked in synchronous part of watchpoint code 
(Watchpoint::ShouldStop), so that 
we avoid pulling an event even if watchpoint fails ignore count condition. It 
is moved here temporarily, because for
archs with watchpoint_exceptions_received=before, there is an adjustment code 
in above lines, which takes control 
of inferior to next PC. We have to check ignore count condition after this is 
done, otherwise we will get same watchpoint 
multiple  times untill we pass ignore condition, and we won't be ignoring them 
actually.


Repository:
  rL LLVM

http://reviews.llvm.org/D13296



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to