[Lldb-commits] [lldb] [lldb] Add 'modify' type watchpoints, make it default (PR #66308)

[Alex Langford via lldb-commits]

bulbazord wrote:

> > ```
> >    SBWatchpoint SBTarget::WatchpointCreateByAddress(addr_t address, size_t size, uint32_t access_flags, SBError &error);
> > ```
> > 
> > 
> >     
> >       
> >     
> > 
> >       
> >     
> > 
> >     
> >   
> > with `eWatchpointAccess{Read,Write,Modify}` flags defined.
> 
> @bulbazord what do you think about this suggestion? Would you still prefer an Options class?

I'm alright with an Options class. What I would like to see is flexibility instead of adding new overloads every time we may want to add new levers and knobs to change Watchpoint behavior.

https://github.com/llvm/llvm-project/pull/66308