Skip to content
  • Tankut Baris Aktemur's avatar
    gdb/breakpoint: add flags to 'condition' and 'break' commands to force condition · 733d554a
    Tankut Baris Aktemur authored
    The previous patch made it possible to define a condition if it's
    valid at some locations.  If the condition is invalid at all of the
    locations, it's rejected.  However, there may be cases where the user
    knows the condition *will* be valid at a location in the future,
    e.g. due to a shared library load.
    
    To make it possible that such condition can be defined, this patch
    adds an optional '-force' flag to the 'condition' command, and,
    respectively, a '-force-condition' flag to the 'break'command.  When
    the force flag is passed, the condition is not rejected even when it
    is invalid for all the current locations (note that all the locations
    would be internally disabled in this case).
    
    For instance:
    
      (gdb) break test.c:5
      Breakpoint 1 at 0x1155: file test.c, line 5.
      (gdb) cond 1 foo == 42
      No symbol "foo" in current context.
    
    Defining the condition was not possible because 'foo' is not
    available.  The user can override this behavior with the '-force'
    flag:
    
      (gdb) cond -force 1 foo == 42
      warning: failed to validate condition at location 1.1, disabling:
        No symbol "foo" in current context.
      (gdb) info breakpoints
      Num     Type           Disp Enb Address            What
      1       breakpoint     keep y   <MULTIPLE>
              stop only if foo == 42
      1.1                         N   0x0000000000001155 in main at test.c:5
    
    Now the condition is accepted, but the location is automatically
    disabled.  If a future location has a context in which 'foo' is
    available, that location would be enabled.
    
    For the 'break' command, -force-condition has the same result:
    
      (gdb) break test.c:5 -force-condition if foo == 42
      warning: failed to validate condition at location 0x1169, disabling:
        No symbol "foo" in current context.
      Breakpoint 1 at 0x1169: file test.c, line 5.
    
    gdb/ChangeLog:
    2020-10-27  Tankut Baris Aktemur  <tankut.baris.aktemur@intel.com>
    
    	* breakpoint.h (set_breakpoint_condition): Add a new bool parameter.
    	* breakpoint.c: Update the help text of the 'condition' and 'break'
    	commands.
    	(set_breakpoint_condition): Take a new bool parameter
    	to control whether condition definition should be forced even when
    	the condition expression is invalid in all of the current locations.
    	(condition_command): Update the call to 'set_breakpoint_condition'.
    	(find_condition_and_thread): Take the "-force-condition" flag into
    	account.
            * linespec.c (linespec_keywords): Add "-force-condition" as an
    	element.
            (FORCE_KEYWORD_INDEX): New #define.
            (linespec_lexer_lex_keyword): Update to consider "-force-condition"
    	as a keyword.
    	* ada-lang.c (create_ada_exception_catchpoint): Ditto.
    	* guile/scm-breakpoint.c (gdbscm_set_breakpoint_condition_x): Ditto.
    	* python/py-breakpoint.c (bppy_set_condition): Ditto.
    	* NEWS: Mention the changes to the 'break' and 'condition' commands.
    
    gdb/testsuite/ChangeLog:
    2020-10-27  Tankut Baris Aktemur  <tankut.baris.aktemur@intel.com>
    
    	* gdb.base/condbreak-multi-context.exp: Expand to test forcing
    	the condition.
    	* gdb.linespec/cpcompletion.exp: Update to consider the
    	'-force-condition' keyword.
    	* gdb.linespec/explicit.exp: Ditto.
    	* lib/completion-support.exp: Ditto.
    
    gdb/doc/ChangeLog:
    2020-10-27  Tankut Baris Aktemur  <tankut.baris.aktemur@intel.com>
    
    	* gdb.texinfo (Set Breaks): Document the '-force-condition' flag
    	of the 'break'command.
    	* gdb.texinfo (Conditions): Document the '-force' flag of the
    	'condition' command.
    733d554a