[gdb/contrib] Add two rules in common-misspellings.txt

Eli mentioned [1] that given that we use US English spelling in our
documentation, we should use "behavior" instead of "behaviour".

In wikipedia-common-misspellings.txt there's a rule:
...
behavour->behavior, behaviour
...
which leaves this as a choice.

Add an overriding rule to hardcode the choice to common-misspellings.txt:
...
behavour->behavior
...
and add a rule to rewrite behaviour into behavior:
...
behaviour->behavior
...
and re-run spellcheck.sh on gdb*.

Tested on x86_64-linux.

[1] https://sourceware.org/pipermail/gdb-patches/2024-November/213371.html

80 files changed, 131 insertions, 129 deletions

diff --git a/gdb/NEWS b/gdb/NEWS
index f3c5d720..8cb35f4 100644
--- a/gdb/NEWS
+++ b/gdb/NEWS
@@ -605,7 +605,7 @@ show tui mouse-events
      changed.
 
   ** New methods added to the gdb.PendingFrame class.  These methods
-     have the same behaviour as the corresponding methods on
+     have the same behavior as the corresponding methods on
      gdb.Frame.  The new methods are:
 
      - gdb.PendingFrame.name: Return the name for the frame's
@@ -1231,7 +1231,7 @@ winheight
 * MI changes
 
  ** The '-add-inferior' with no option flags now inherits the
-    connection of the current inferior, this restores the behaviour of
+    connection of the current inferior, this restores the behavior of
     GDB as it was prior to GDB 10.
 
  ** The '-add-inferior' command now accepts a '--no-connection'
@@ -2012,7 +2012,7 @@ set print frame-info [short-location|location|location-and-address
                         |source-and-location|source-line|auto]
 show print frame-info
   This controls what frame information is printed by the commands printing
-  a frame.  This setting will e.g. influence the behaviour of 'backtrace',
+  a frame.  This setting will e.g. influence the behavior of 'backtrace',
   'frame', 'stepi'.  The python frame filtering also respect this setting.
   The 'backtrace' '-frame-info' option can override this global setting.
 
@@ -8098,7 +8098,7 @@ full 64-bit address.  The command
 
 	set remoteaddresssize 32
 
-can be used to revert to the old behaviour.  For existing remote stubs
+can be used to revert to the old behavior.  For existing remote stubs
 the change should not be noticed, as the additional address information
 will be discarded.
 
diff --git a/gdb/SECURITY.txt b/gdb/SECURITY.txt
index 0657e1e..008bf88 100644
--- a/gdb/SECURITY.txt
+++ b/gdb/SECURITY.txt
@@ -135,7 +135,7 @@ What Is Not A Security Bug
   It is possible for a program to detect when it is run under GDB and
   to change its behavior so that unwanted behavior may only appear
   when a program is run under GDB.  Any issues that arise due to an
-  untrusted program detecting GDB and changing its behaviour are not
+  untrusted program detecting GDB and changing its behavior are not
   security issues in GDB unless the issue also meet some other
   definition of a security bug.
 
@@ -191,7 +191,7 @@ Security Realities Of The GDB Project
 
   There are known bugs in GDB related to loading malformed executables
   and parsing the debug information, a consequence of these bugs is
-  that a malicious program could trigger undefined behaviour in GDB,
+  that a malicious program could trigger undefined behavior in GDB,
   which could be used to trigger arbitrary code execution.
 
   Given these risks, the advice of the GDB project is that, when using
@@ -199,7 +199,7 @@ Security Realities Of The GDB Project
   environment.
 
   As there are already known bugs in GDB relating to undefined
-  behaviour triggered from malformed programs, further bugs in this
+  behavior triggered from malformed programs, further bugs in this
   area should still be reported, but are unlikely to be given high
   priority.  Bugs in GDB that are triggered by well-formed programs
   should also be reported, and are likely to be treated as higher
diff --git a/gdb/aarch64-tdep.c b/gdb/aarch64-tdep.c
index 6850baf..bc8746e 100644
--- a/gdb/aarch64-tdep.c
+++ b/gdb/aarch64-tdep.c
@@ -2584,7 +2584,7 @@ aarch64_extract_return_value (struct type *type, struct regcache *regs,
     }
   else
     {
-      /* For a structure or union the behaviour is as if the value had
+      /* For a structure or union the behavior is as if the value had
 	 been stored to word-aligned memory and then loaded into
 	 registers with 64-bit load instruction(s).  */
       int len = type->length ();
@@ -2710,7 +2710,7 @@ aarch64_store_return_value (struct type *type, struct regcache *regs,
     }
   else
     {
-      /* For a structure or union the behaviour is as if the value had
+      /* For a structure or union the behavior is as if the value had
 	 been stored to word-aligned memory and then loaded into
 	 registers with 64-bit load instruction(s).  */
       int len = type->length ();
diff --git a/gdb/amd64-tdep.c b/gdb/amd64-tdep.c
index ba6532f..e03180b 100644
--- a/gdb/amd64-tdep.c
+++ b/gdb/amd64-tdep.c
@@ -1714,7 +1714,7 @@ amd64_displaced_step_fixup (struct gdbarch *gdbarch,
 	  CORE_ADDR rip = pc - insn_offset;
 
 	  /* If we just stepped over a breakpoint insn, we don't backup
-	     the pc on purpose; this is to match behaviour without
+	     the pc on purpose; this is to match behavior without
 	     stepping.  */
 
 	  regcache_write_pc (regs, rip);
diff --git a/gdb/annotate.h b/gdb/annotate.h
index db471db..27a35db 100644
--- a/gdb/annotate.h
+++ b/gdb/annotate.h
@@ -112,7 +112,7 @@ struct annotate_arg_emitter
    are on, we only sometimes print the annotation, and only sometimes
    update the current symtab and line.  However, this particular annotation
    has behaved this way for some time, and front ends that still use
-   annotations now depend on this behaviour.  */
+   annotations now depend on this behavior.  */
 extern bool annotate_source_line (struct symtab *s, int line,
 				  int mid_statement, CORE_ADDR pc);
 
diff --git a/gdb/arch-utils.c b/gdb/arch-utils.c
index 6ffa410..a2ed2a2 100644
--- a/gdb/arch-utils.c
+++ b/gdb/arch-utils.c
@@ -75,7 +75,7 @@ legacy_register_sim_regno (struct gdbarch *gdbarch, int regnum)
   gdb_assert (regnum >= 0 && regnum < gdbarch_num_regs (gdbarch));
   /* NOTE: cagney/2002-05-13: The old code did it this way and it is
      suspected that some GDB/SIM combinations may rely on this
-     behaviour.  The default should be one2one_register_sim_regno
+     behavior.  The default should be one2one_register_sim_regno
      (below).  */
   if (gdbarch_register_name (gdbarch, regnum)[0] != '\0')
     return regnum;
diff --git a/gdb/arm-tdep.c b/gdb/arm-tdep.c
index 3450761..c27b2a2 100644
--- a/gdb/arm-tdep.c
+++ b/gdb/arm-tdep.c
@@ -588,7 +588,7 @@ static CORE_ADDR arm_analyze_prologue
   (struct gdbarch *gdbarch, CORE_ADDR prologue_start, CORE_ADDR prologue_end,
    struct arm_prologue_cache *cache, const arm_instruction_reader &insn_reader);
 
-/* Architecture version for displaced stepping.  This effects the behaviour of
+/* Architecture version for displaced stepping.  This effects the behavior of
    certain instructions, and really should not be hard-wired.  */
 
 #define DISPLACED_STEPPING_ARCH_VERSION		5
@@ -5574,7 +5574,7 @@ bx_write_pc (struct regcache *regs, ULONGEST val)
     }
   else
     {
-      /* Unpredictable behaviour.  Try to do something sensible (switch to ARM
+      /* Unpredictable behavior.  Try to do something sensible (switch to ARM
 	  mode, align dest to 4 bytes).  */
       warning (_("Single-stepping BX to non-word-aligned ARM instruction."));
       regcache_cooked_write_unsigned (regs, ARM_PS_REGNUM, ps & ~t_bit);
@@ -8776,7 +8776,7 @@ gdb_print_insn_arm (bfd_vma memaddr, disassemble_info *info)
    undefined instruction trap.  ARM7TDMI is nominally ARMv4T, but does
    not in fact add the new instructions.  The new undefined
    instructions in ARMv4 are all instructions that had no defined
-   behaviour in earlier chips.  There is no guarantee that they will
+   behavior in earlier chips.  There is no guarantee that they will
    raise an exception, but may be treated as NOP's.  In practice, it
    may only safe to rely on instructions matching:
    
@@ -8983,7 +8983,7 @@ arm_extract_return_value (struct type *type, struct regcache *regs,
     }
   else
     {
-      /* For a structure or union the behaviour is as if the value had
+      /* For a structure or union the behavior is as if the value had
 	 been stored to word-aligned memory and then loaded into 
 	 registers with 32-bit load instruction(s).  */
       int len = type->length ();
@@ -9058,7 +9058,7 @@ arm_return_in_memory (struct gdbarch *gdbarch, struct type *type)
 	 fields are not addressable, and all addressable subfields of
 	 unions always start at offset zero.
 
-	 This function is based on the behaviour of GCC 2.95.1.
+	 This function is based on the behavior of GCC 2.95.1.
 	 See: gcc/arm.c: arm_return_in_memory() for details.
 
 	 Note: All versions of GCC before GCC 2.95.2 do not set up the
@@ -9213,7 +9213,7 @@ arm_store_return_value (struct type *type, struct regcache *regs,
     }
   else
     {
-      /* For a structure or union the behaviour is as if the value had
+      /* For a structure or union the behavior is as if the value had
 	 been stored to word-aligned memory and then loaded into 
 	 registers with 32-bit load instruction(s).  */
       int len = type->length ();
diff --git a/gdb/auto-load.c b/gdb/auto-load.c
index d4307ea..e684d7f 100644
--- a/gdb/auto-load.c
+++ b/gdb/auto-load.c
@@ -787,7 +787,7 @@ auto_load_objfile_script_1 (struct objfile *objfile, const char *realname,
       maybe_add_script_file (pspace_info, is_safe, debugfile, debugfile,
 			     language);
 
-      /* To preserve existing behaviour we don't check for whether the
+      /* To preserve existing behavior we don't check for whether the
 	 script was already in the table, and always load it.
 	 It's highly unlikely that we'd ever load it twice,
 	 and these scripts are required to be idempotent under multiple
diff --git a/gdb/break-cond-parse.c b/gdb/break-cond-parse.c
index b2b1324..0428cdf 100644
--- a/gdb/break-cond-parse.c
+++ b/gdb/break-cond-parse.c
@@ -368,7 +368,7 @@ parse_all_tokens (const char *str)
      which exists for backwards compatibility reasons.  Traditionally this
      flag, if it occurred immediately after the 'if' condition, would be
      treated as part of the 'if' condition.  When the breakpoint condition
-     parsing code was rewritten, this behaviour was retained.  */
+     parsing code was rewritten, this behavior was retained.  */
   gdb_assert (backward_results.empty ()
 	      || (forward_results.back ().get_type ()
 		  == token::type::CONDITION));
diff --git a/gdb/c-lang.c b/gdb/c-lang.c
index 24cdde6..c28493f 100644
--- a/gdb/c-lang.c
+++ b/gdb/c-lang.c
@@ -337,17 +337,17 @@ c_get_string (struct value *value, gdb::unique_xmalloc_ptr<gdb_byte> *buffer,
 	addr = value_as_address (value);
 
       /* Prior to the fix for PR 16196 read_string would ignore fetchlimit
-	 if length > 0.  The old "broken" behaviour is the behaviour we want:
+	 if length > 0.  The old "broken" behavior is the behavior we want:
 	 The caller may want to fetch 100 bytes from a variable length array
 	 implemented using the common idiom of having an array of length 1 at
 	 the end of a struct.  In this case we want to ignore the declared
 	 size of the array.  However, it's counterintuitive to implement that
-	 behaviour in read_string: what does fetchlimit otherwise mean if
-	 length > 0.  Therefore we implement the behaviour we want here:
+	 behavior in read_string: what does fetchlimit otherwise mean if
+	 length > 0.  Therefore we implement the behavior we want here:
 	 If *length > 0, don't specify a fetchlimit.  This preserves the
-	 previous behaviour.  We could move this check above where we know
+	 previous behavior.  We could move this check above where we know
 	 whether the array is declared with a fixed size, but we only want
-	 to apply this behaviour when calling read_string.  PR 16286.  */
+	 to apply this behavior when calling read_string.  PR 16286.  */
       if (*length > 0)
 	fetchlimit = UINT_MAX;
 
diff --git a/gdb/configure b/gdb/configure
index ec9bbd3..fcebd3e 100755
--- a/gdb/configure
+++ b/gdb/configure
@@ -27961,7 +27961,7 @@ else
         python_prefix=
       fi
       if test ! -x "${python_prog}"; then
-        # Fall back to gdb 7.0/7.1 behaviour.
+        # Fall back to gdb 7.0/7.1 behavior.
         python_prog=missing
         python_prefix=${with_python}
       fi
diff --git a/gdb/configure.ac b/gdb/configure.ac
index 21f5dc8..6179b5c 100644
--- a/gdb/configure.ac
+++ b/gdb/configure.ac
@@ -802,7 +802,7 @@ else
         python_prefix=
       fi
       if test ! -x "${python_prog}"; then
-        # Fall back to gdb 7.0/7.1 behaviour.
+        # Fall back to gdb 7.0/7.1 behavior.
         python_prog=missing
         python_prefix=${with_python}
       fi
diff --git a/gdb/contrib/common-misspellings.txt b/gdb/contrib/common-misspellings.txt
index 11ca8ec..4ab968e 100644
--- a/gdb/contrib/common-misspellings.txt
+++ b/gdb/contrib/common-misspellings.txt
@@ -20,6 +20,8 @@
 inbetween->between, in between, in-between
 sofar->so far
 doens't->doesn't
+behavour->behavior
+behaviour->behavior
 
 # Identity rules.
 
diff --git a/gdb/contrib/gdb-add-index.sh b/gdb/contrib/gdb-add-index.sh
index bd5a8f7..00a9bea 100755
--- a/gdb/contrib/gdb-add-index.sh
+++ b/gdb/contrib/gdb-add-index.sh
@@ -122,7 +122,7 @@ $GDB --batch -nx -iex 'set auto-load no' \
 
 # In some situations gdb can exit without creating an index.  This is
 # not an error.
-# E.g., if $file is stripped.  This behaviour is akin to stripping an
+# E.g., if $file is stripped.  This behavior is akin to stripping an
 # already stripped binary, it's a no-op.
 status=0
 
diff --git a/gdb/corelow.c b/gdb/corelow.c
index 25de755..c176b24 100644
--- a/gdb/corelow.c
+++ b/gdb/corelow.c
@@ -1523,7 +1523,7 @@ core_target::xfer_partial (enum target_object object, const char *annex,
    exactly lively, are they?  On the other hand, if we don't claim
    that each & every one is alive, then we don't get any of them
    to appear in an "info thread" command, which is quite a useful
-   behaviour.
+   behavior.
  */
 bool
 core_target::thread_alive (ptid_t ptid)
diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo
index b91b5d6..0383761 100644
--- a/gdb/doc/gdb.texinfo
+++ b/gdb/doc/gdb.texinfo
@@ -4209,7 +4209,7 @@ call executes, the new target restarts.  To restart the parent
 process, use the @code{file} command with the parent executable name
 as its argument.  By default, after an @code{exec} call executes,
 @value{GDBN} discards the symbols of the previous executable image.
-You can change this behaviour with the @w{@code{set follow-exec-mode}}
+You can change this behavior with the @w{@code{set follow-exec-mode}}
 command.
 
 @table @code
@@ -4977,7 +4977,7 @@ breakpoints set by commands like @code{next} and @code{finish}.  For
 breakpoints set with @code{hbreak}, @value{GDBN} will always use hardware
 breakpoints.
 
-You can control this automatic behaviour with the following commands:
+You can control this automatic behavior with the following commands:
 
 @kindex set breakpoint auto-hw
 @kindex show breakpoint auto-hw
@@ -4998,7 +4998,7 @@ at the breakpoint address with a special instruction, which, when
 executed, given control to the debugger.  By default, the program
 code is so modified only when the program is resumed.  As soon as
 the program stops, @value{GDBN} restores the original instructions.  This
-behaviour guards against leaving breakpoints inserted in the
+behavior guards against leaving breakpoints inserted in the
 target should gdb abrubptly disconnect.  However, with slow remote
 targets, inserting and removing breakpoint can reduce the performance.
 This behavior can be controlled with the following commands::
@@ -8177,7 +8177,7 @@ are printed in execution order.
 It can also print mixed source+disassembly if you specify the the
 @code{/m} or @code{/s} modifier, and print the raw instructions in hex
 as well as in symbolic form by specifying the @code{/r} or @code{/b}
-modifier.  The behaviour of the @code{/m}, @code{/s}, @code{/r}, and
+modifier.  The behavior of the @code{/m}, @code{/s}, @code{/r}, and
 @code{/b} modifiers are the same as for the @kbd{disassemble} command
 (@pxref{disassemble,,@kbd{disassemble}}).
 
@@ -13805,7 +13805,7 @@ Disable @value{GDBN} from caching target memory.  This is the default.
 @value{GDBN} can be instructed to refuse accesses to memory that is
 not explicitly described.  This can be useful if accessing such
 regions has undesired effects for a specific target, or to provide
-better error checking.  The following commands control this behaviour.
+better error checking.  The following commands control this behavior.
 
 @table @code
 @kindex set mem inaccessible-by-default
@@ -17593,7 +17593,7 @@ Because of a limitation in @file{libdecnumber}, the library used by @value{GDBN}
 to manipulate decimal floating point numbers, it is not possible to convert
 (using a cast, for example) integers wider than 32-bit to decimal float.
 
-In addition, in order to imitate @value{GDBN}'s behaviour with binary floating
+In addition, in order to imitate @value{GDBN}'s behavior with binary floating
 point computations, error checking in decimal float operations ignores
 underflow, overflow and divide by zero exceptions.
 
@@ -21120,7 +21120,7 @@ still ongoing, then @value{GDBN} will interrupt the program.
 
 If a function called from @value{GDBN} is interrupted by a timeout,
 then by default the inferior is left in the frame where the timeout
-occurred, this behaviour can be adjusted with @samp{set
+occurred, this behavior can be adjusted with @samp{set
 unwind-on-timeout} (@pxref{set unwind-on-timeout}).
 
 For targets that don't support asynchronous execution
@@ -30296,7 +30296,7 @@ There are some things to be aware of, however.
 @subsection Python comes first
 
 Python was @value{GDBN}'s first extension language, and to avoid breaking
-existing behaviour Python comes first.  This is generally solved by the
+existing behavior Python comes first.  This is generally solved by the
 ``first one wins'' principle.  @value{GDBN} maintains a list of enabled
 extension languages, and when it makes a call to an extension language,
 (say to pretty-print a value), it tries each in turn until an extension
@@ -31415,7 +31415,7 @@ if the selected thread in @value{GDBN} is supposed to be identical to the
 thread the frontend wants to operate on.  However, getting this
 optimization right can be tricky.  In particular, if the frontend
 sends several commands to @value{GDBN}, and one of the commands changes the
-selected thread, then the behaviour of subsequent commands will
+selected thread, then the behavior of subsequent commands will
 change.  So, a frontend should either wait for response from such
 problematic commands, or explicitly add @code{-thread-select} for
 all subsequent commands.  No frontend is known to do this exactly
@@ -31782,7 +31782,7 @@ details about the various output records.
 @cindex @sc{gdb/mi}, compatibility with CLI
 
 For the developers convenience CLI commands can be entered directly,
-but there may be some unexpected behaviour.  For example, commands
+but there may be some unexpected behavior.  For example, commands
 that query the user will behave as if the user replied yes, breakpoint
 command lists are not executed and some CLI commands, such as
 @code{if}, @code{when} and @code{define}, prompt for further input with
@@ -31961,7 +31961,7 @@ values.
 @item "^running"
 This result record is equivalent to @samp{^done}.  Historically, it
 was output instead of @samp{^done} if the command has resumed the
-target.  This behaviour is maintained for backward compatibility, but
+target.  This behavior is maintained for backward compatibility, but
 all frontends should treat @samp{^done} and @samp{^running}
 identically and rely on the @samp{*running} output record to determine
 which threads are resumed.
@@ -41702,7 +41702,7 @@ in @value{GDBN} downloading larger amounts of data.
 When @value{GDBN} reports an internal problem (error or warning) it
 gives the user the opportunity to both quit @value{GDBN} and create a
 core file of the current @value{GDBN} session.  These commands let you
-override the default behaviour for each particular @var{action},
+override the default behavior for each particular @var{action},
 described in the table below.
 
 @table @samp
@@ -47123,7 +47123,7 @@ conditions is met:
 
 @itemize @bullet
 @item
-The user types @kbd{Ctrl-c}.  The behaviour is as explained above, and the
+The user types @kbd{Ctrl-c}.  The behavior is as explained above, and the
 @code{read}
 system call is treated as finished.
 
diff --git a/gdb/doc/python.texi b/gdb/doc/python.texi
index cc45548..5cbcc5b 100644
--- a/gdb/doc/python.texi
+++ b/gdb/doc/python.texi
@@ -1180,7 +1180,7 @@ refer to, @code{False} (the default) if they shouldn't.  Note that, unlike
 for the @code{print} command, references are not automatically expanded
 when using the @code{format_string} method or the @code{str}
 function.  There is no global @code{print} setting to change the default
-behaviour.
+behavior.
 
 @item actual_objects
 @code{True} if the representation of a pointer to an object should
diff --git a/gdb/dwarf2/frame.c b/gdb/dwarf2/frame.c
index 841d2d4..6d8a4fb 100644
--- a/gdb/dwarf2/frame.c
+++ b/gdb/dwarf2/frame.c
@@ -2043,7 +2043,7 @@ decode_frame_entry (struct gdbarch *gdbarch,
 	 produces a hole in the frame info that gets filled by the 
 	 linker with zeros.
 
-	 The GCC behaviour is arguably a bug, but it's effectively now
+	 The GCC behavior is arguably a bug, but it's effectively now
 	 part of the ABI, so we're now stuck with it, at least at the
 	 object file level.  A smart linker may decide, in the process
 	 of compressing duplicate CIE information, that it can rewrite
diff --git a/gdb/event-top.c b/gdb/event-top.c
index cab6c84..45ad7b9 100644
--- a/gdb/event-top.c
+++ b/gdb/event-top.c
@@ -1031,7 +1031,7 @@ handle_fatal_signal (int sig)
     }
 #endif
 
-  /* If possible arrange for SIG to have its default behaviour (which
+  /* If possible arrange for SIG to have its default behavior (which
      should be to terminate the current process), unblock SIG, and reraise
      the signal.  This ensures GDB terminates with the expected signal.  */
   if (signal (sig, SIG_DFL) != SIG_ERR
diff --git a/gdb/extension.c b/gdb/extension.c
index 47f5691..f13d391 100644
--- a/gdb/extension.c
+++ b/gdb/extension.c
@@ -85,7 +85,7 @@ const struct extension_language_defn extension_language_gdb =
 
 static const std::array<const extension_language_defn *, 2> extension_languages
 {
-  /* To preserve existing behaviour, python should always appear first.  */
+  /* To preserve existing behavior, python should always appear first.  */
   &extension_language_python,
   &extension_language_guile,
 };
diff --git a/gdb/extension.h b/gdb/extension.h
index ad3b4e6..793a66f 100644
--- a/gdb/extension.h
+++ b/gdb/extension.h
@@ -177,7 +177,7 @@ enum ext_lang_rc
 
   /* There was an error (e.g., Python error while printing a value).
      When an error occurs no further extension languages are tried.
-     This is to preserve existing behaviour, and because it's convenient
+     This is to preserve existing behavior, and because it's convenient
      for Python developers.
      Note: This is different than encountering a memory error trying to read
      a value for pretty-printing.  Here we're referring to, e.g., programming
diff --git a/gdb/f-array-walker.h b/gdb/f-array-walker.h
index bcbaa3d..7282d5f 100644
--- a/gdb/f-array-walker.h
+++ b/gdb/f-array-walker.h
@@ -95,7 +95,7 @@ private:
 
 /* A base class used by fortran_array_walker.  There's no virtual methods
    here, sub-classes should just override the functions they want in order
-   to specialise the behaviour to their needs.  The functionality
+   to specialise the behavior to their needs.  The functionality
    provided in these default implementations will visit every array
    element, but do nothing for each element.  */
 
diff --git a/gdb/gdbthread.h b/gdb/gdbthread.h
index 73f6895..1011cf8 100644
--- a/gdb/gdbthread.h
+++ b/gdb/gdbthread.h
@@ -216,7 +216,7 @@ struct thread_suspend_state
      - If the thread is running, then this field has its value removed by
        calling stop_pc.reset() (see thread_info::set_executing()).
        Attempting to read a std::optional with no value is undefined
-       behaviour and will trigger an assertion error when _GLIBCXX_DEBUG is
+       behavior and will trigger an assertion error when _GLIBCXX_DEBUG is
        defined, which should make error easier to track down.  */
   std::optional<CORE_ADDR> stop_pc;
 };
diff --git a/gdb/guile/scm-auto-load.c b/gdb/guile/scm-auto-load.c
index 57550fa..ea87ab7 100644
--- a/gdb/guile/scm-auto-load.c
+++ b/gdb/guile/scm-auto-load.c
@@ -61,8 +61,8 @@ gdbscm_initialize_auto_load (void)
 {
   add_setshow_boolean_cmd ("guile-scripts", class_support,
 			   &auto_load_guile_scripts, _("\
-Set the debugger's behaviour regarding auto-loaded Guile scripts."), _("\
-Show the debugger's behaviour regarding auto-loaded Guile scripts."), _("\
+Set the debugger's behavior regarding auto-loaded Guile scripts."), _("\
+Show the debugger's behavior regarding auto-loaded Guile scripts."), _("\
 If enabled, auto-loaded Guile scripts are loaded when the debugger reads\n\
 an executable or shared library.\n\
 This options has security implications for untrusted inferiors."),
diff --git a/gdb/guile/scm-lazy-string.c b/gdb/guile/scm-lazy-string.c
index 94032ae..9f525fa 100644
--- a/gdb/guile/scm-lazy-string.c
+++ b/gdb/guile/scm-lazy-string.c
@@ -206,7 +206,7 @@ lsscm_elt_type (lazy_string_smob *ls_smob)
     case TYPE_CODE_ARRAY:
       return realtype->target_type ();
     default:
-      /* This is done to preserve existing behaviour.  PR 20769.
+      /* This is done to preserve existing behavior.  PR 20769.
 	 E.g., gdb.parse_and_eval("my_int_variable").lazy_string().type.  */
       return realtype;
     }
diff --git a/gdb/i386-tdep.c b/gdb/i386-tdep.c
index eb8ddfc..a552a2b 100644
--- a/gdb/i386-tdep.c
+++ b/gdb/i386-tdep.c
@@ -866,7 +866,7 @@ i386_displaced_step_fixup (struct gdbarch *gdbarch,
 	  ULONGEST eip = (pc - insn_offset) & 0xffffffffUL;
 
 	  /* If we just stepped over a breakpoint insn, we don't backup
-	     the pc on purpose; this is to match behaviour without
+	     the pc on purpose; this is to match behavior without
 	     stepping.  */
 
 	  regcache_write_pc (regs, eip);
diff --git a/gdb/infcall.c b/gdb/infcall.c
index 02e3b38..6399278 100644
--- a/gdb/infcall.c
+++ b/gdb/infcall.c
@@ -1834,7 +1834,7 @@ The program being debugged entered a std::terminate call, most likely\n\
 caused by an unhandled C++ exception.  GDB blocked this call in order\n\
 to prevent the program from being terminated, and has restored the\n\
 context to its original state before the call.\n\
-To change this behaviour use \"set unwind-on-terminating-exception off\".\n\
+To change this behavior use \"set unwind-on-terminating-exception off\".\n\
 Evaluation of the expression containing the function (%s)\n\
 will be abandoned."),
 		 name.c_str ());
diff --git a/gdb/infrun.c b/gdb/infrun.c
index 623c2d3..43eca81 100644
--- a/gdb/infrun.c
+++ b/gdb/infrun.c
@@ -4958,7 +4958,7 @@ adjust_pc_after_break (struct thread_info *thread,
      we would wrongly adjust the PC to 0x08000000, since there's a
      breakpoint at PC - 1.  We'd then report a hit on B1, although
      INSN1 hadn't been de-executed yet.  Doing nothing is the correct
-     behaviour.  */
+     behavior.  */
   if (execution_direction == EXEC_REVERSE)
     return;
 
diff --git a/gdb/language.h b/gdb/language.h
index 26d7fb2..6653f99 100644
--- a/gdb/language.h
+++ b/gdb/language.h
@@ -788,7 +788,7 @@ struct symbol *
 					    const char *name);
 
 
-/* These macros define the behaviour of the expression 
+/* These macros define the behavior of the expression 
    evaluator.  */
 
 /* Should we range check values against the domain of their type?  */
diff --git a/gdb/location.c b/gdb/location.c
index 851d6e4..1b40991 100644
--- a/gdb/location.c
+++ b/gdb/location.c
@@ -78,7 +78,7 @@ linespec_location_spec::linespec_location_spec
       p = remove_trailing_whitespace (orig, *linespec);
 
       /* If there is no valid linespec then this will leave the
-	 spec_string as nullptr.  This behaviour is relied on in the
+	 spec_string as nullptr.  This behavior is relied on in the
 	 breakpoint setting code, where spec_string being nullptr means
 	 to use the default breakpoint location.  */
       if ((p - orig) > 0)
diff --git a/gdb/mdebugread.c b/gdb/mdebugread.c
index dae4e4d..fbfe3a7 100644
--- a/gdb/mdebugread.c
+++ b/gdb/mdebugread.c
@@ -4401,7 +4401,7 @@ cross_ref (int fd, union aux_ext *ax, struct type **tpp,
 	    case btTypedef:
 	      /* Follow a forward typedef.  This might recursively
 		 call cross_ref till we get a non typedef'ed type.
-		 FIXME: This is not correct behaviour, but gdb currently
+		 FIXME: This is not correct behavior, but gdb currently
 		 cannot handle typedefs without type copying.  Type
 		 copying is impossible as we might have mutual forward
 		 references between two files and the copied type would not
@@ -4427,7 +4427,7 @@ cross_ref (int fd, union aux_ext *ax, struct type **tpp,
 	{
 	  /* Parse the type for a normal typedef.  This might recursively call
 	     cross_ref till we get a non typedef'ed type.
-	     FIXME: This is not correct behaviour, but gdb currently
+	     FIXME: This is not correct behavior, but gdb currently
 	     cannot handle typedefs without type copying.  But type copying is
 	     impossible as we might have mutual forward references between
 	     two files and the copied type would not get filled in when
diff --git a/gdb/mi/mi-main.c b/gdb/mi/mi-main.c
index b72cd1b9..1d0fae9 100644
--- a/gdb/mi/mi-main.c
+++ b/gdb/mi/mi-main.c
@@ -2077,7 +2077,7 @@ mi_cmd_execute (struct mi_parse *parse)
 	error (_("Invalid thread group for the --thread-group option"));
 
       set_current_inferior (inf);
-      /* This behaviour means that if --thread-group option identifies
+      /* This behavior means that if --thread-group option identifies
 	 an inferior with multiple threads, then a random one will be
 	 picked.  This is not a problem -- frontend should always
 	 provide --thread if it wishes to operate on a specific
diff --git a/gdb/opencl-lang.c b/gdb/opencl-lang.c
index 2132778..939757d 100644
--- a/gdb/opencl-lang.c
+++ b/gdb/opencl-lang.c
@@ -581,7 +581,7 @@ vector_relop (struct expression *exp, struct value *val1, struct value *val2,
 
 /* Perform a cast of ARG into TYPE.  There's sadly a lot of duplication in
    here from valops.c:value_cast, opencl is different only in the
-   behaviour of scalar to vector casting.  As far as possibly we're going
+   behavior of scalar to vector casting.  As far as possibly we're going
    to try and delegate back to the standard value_cast function. */
 
 struct value *
diff --git a/gdb/python/py-auto-load.c b/gdb/python/py-auto-load.c
index 9548973..7dc3a7b 100644
--- a/gdb/python/py-auto-load.c
+++ b/gdb/python/py-auto-load.c
@@ -61,8 +61,8 @@ gdbpy_initialize_auto_load (void)
 {
   add_setshow_boolean_cmd ("python-scripts", class_support,
 			   &auto_load_python_scripts, _("\
-Set the debugger's behaviour regarding auto-loaded Python scripts."), _("\
-Show the debugger's behaviour regarding auto-loaded Python scripts."), _("\
+Set the debugger's behavior regarding auto-loaded Python scripts."), _("\
+Show the debugger's behavior regarding auto-loaded Python scripts."), _("\
 If enabled, auto-loaded Python scripts are loaded when the debugger reads\n\
 an executable or shared library.\n\
 This options has security implications for untrusted inferiors."),
@@ -73,10 +73,10 @@ This options has security implications for untrusted inferiors."),
   set_show_commands auto_load_scripts_cmds
     = add_setshow_boolean_cmd ("auto-load-scripts", class_support,
 			       &auto_load_python_scripts, _("\
-Set the debugger's behaviour regarding auto-loaded Python scripts, "
+Set the debugger's behavior regarding auto-loaded Python scripts, "
 								 "deprecated."),
 			       _("\
-Show the debugger's behaviour regarding auto-loaded Python scripts, "
+Show the debugger's behavior regarding auto-loaded Python scripts, "
 								 "deprecated."),
 			       NULL, NULL, show_auto_load_python_scripts,
 			       &setlist, &showlist);
diff --git a/gdb/python/py-connection.c b/gdb/python/py-connection.c
index 1fdcd73..7b5146d 100644
--- a/gdb/python/py-connection.c
+++ b/gdb/python/py-connection.c
@@ -342,7 +342,7 @@ struct py_send_packet_callbacks : public send_remote_packet_callbacks
      It is important that the result is inspected immediately after sending
      a packet to the remote, and any error fetched,  calling any other
      Python functions that might clear the error state, or rely on an error
-     not being set will cause undefined behaviour.  */
+     not being set will cause undefined behavior.  */
 
   gdbpy_ref<> result () const
   {
diff --git a/gdb/python/py-disasm.c b/gdb/python/py-disasm.c
index 58e2efe..66ed456 100644
--- a/gdb/python/py-disasm.c
+++ b/gdb/python/py-disasm.c
@@ -308,7 +308,7 @@ disasm_info_dealloc (PyObject *self)
      NEXT is nullptr.  */
   Py_XDECREF ((PyObject *) obj->next);
 
-  /* Now core deallocation behaviour.  */
+  /* Now core deallocation behavior.  */
   Py_TYPE (self)->tp_free (self);
 }
 
@@ -740,7 +740,7 @@ disasmpy_info_progspace (PyObject *self, void *closure)
    part in the gdbpy_disassembler is a text part in the same STYLE, then
    the new string is appended to the previous part.
 
-   The merging behaviour make the Python API a little more user friendly,
+   The merging behavior make the Python API a little more user friendly,
    some disassemblers produce their output character at a time, there's no
    particular reason for this, it's just how they are implemented.  By
    merging parts with the same style we make it easier for the user to
diff --git a/gdb/python/py-framefilter.c b/gdb/python/py-framefilter.c
index daec6dd..f165a48 100644
--- a/gdb/python/py-framefilter.c
+++ b/gdb/python/py-framefilter.c
@@ -762,7 +762,7 @@ py_print_frame (PyObject *filter, frame_filter_flags flags,
      default value for the backtrace command (see the call to print_frame_info
      in backtrace_command_1).
      Having the same default ensures that 'bt' and 'bt no-filters'
-     have the same behaviour when some filters exist but do not apply
+     have the same behavior when some filters exist but do not apply
      to a frame.  */
   enum print_what print_what
     = out->is_mi_like_p () ? LOC_AND_ADDRESS : LOCATION;
diff --git a/gdb/python/py-lazy-string.c b/gdb/python/py-lazy-string.c
index 4898a1f..e61b80a 100644
--- a/gdb/python/py-lazy-string.c
+++ b/gdb/python/py-lazy-string.c
@@ -264,7 +264,7 @@ stpy_lazy_string_elt_type (lazy_string_object *lazy)
     case TYPE_CODE_ARRAY:
       return check_typedef (realtype->target_type ());
     default:
-      /* This is done to preserve existing behaviour.  PR 20769.
+      /* This is done to preserve existing behavior.  PR 20769.
 	 E.g., gdb.parse_and_eval("my_int_variable").lazy_string().type.  */
       return realtype;
     }
diff --git a/gdb/record-btrace.c b/gdb/record-btrace.c
index 49cbf13..bf4a368 100644
--- a/gdb/record-btrace.c
+++ b/gdb/record-btrace.c
@@ -733,7 +733,7 @@ btrace_find_line_range (CORE_ADDR pc)
     {
       /* The test of is_stmt here was added when the is_stmt field was
 	 introduced to the 'struct linetable_entry' structure.  This
-	 ensured that this loop maintained the same behaviour as before we
+	 ensured that this loop maintained the same behavior as before we
 	 introduced is_stmt.  That said, it might be that we would be
 	 better off not checking is_stmt here, this would lead to us
 	 possibly adding more line numbers to the range.  At the time this
diff --git a/gdb/remote-fileio.c b/gdb/remote-fileio.c
index 11cf2bb..fe6ac17 100644
--- a/gdb/remote-fileio.c
+++ b/gdb/remote-fileio.c
@@ -555,7 +555,7 @@ remote_fileio_func_read (remote_target *remote, char *buf)
 	break;
       default:
 	buffer = (gdb_byte *) xmalloc (length);
-	/* POSIX defines EINTR behaviour of read in a weird way.  It's allowed
+	/* POSIX defines EINTR behavior of read in a weird way.  It's allowed
 	   for read() to return -1 even if "some" bytes have been read.  It
 	   has been corrected in SUSv2 but that doesn't help us much...
 	   Therefore a complete solution must check how many bytes have been
diff --git a/gdb/rs6000-tdep.c b/gdb/rs6000-tdep.c
index d7f9698..a36c337 100644
--- a/gdb/rs6000-tdep.c
+++ b/gdb/rs6000-tdep.c
@@ -2163,7 +2163,7 @@ skip_prologue (struct gdbarch *gdbarch, CORE_ADDR pc, CORE_ADDR lim_pc,
      this might be a call to an initializer in main(), introduced by gcc2.
      We'd like to skip over it as well.  Fortunately, xlc does some extra
      work before calling a function right after a prologue, thus we can
-     single out such gcc2 behaviour.  */
+     single out such gcc2 behavior.  */
 
 
   if ((op & 0xfc000001) == 0x48000001)
diff --git a/gdb/source.c b/gdb/source.c
index 3410e86..69480ef 100644
--- a/gdb/source.c
+++ b/gdb/source.c
@@ -758,7 +758,7 @@ prepare_path_for_appending (const char *path)
    using mode MODE in the calls to open.  You cannot use this function to
    create files (O_CREAT).
 
-   OPTS specifies the function behaviour in specific cases.
+   OPTS specifies the function behavior in specific cases.
 
    If OPF_TRY_CWD_FIRST, try to open ./STRING before searching PATH.
    (ie pretend the first element of PATH is ".").  This also indicates
@@ -949,7 +949,7 @@ done:
 }
 
 
-/* This is essentially a convenience, for clients that want the behaviour
+/* This is essentially a convenience, for clients that want the behavior
    of openp, using source_path, but that really don't want the file to be
    opened but want instead just to know what the full pathname is (as
    qualified against source_path).
diff --git a/gdb/symtab.c b/gdb/symtab.c
index 63d2a0e..0aa7321 100644
--- a/gdb/symtab.c
+++ b/gdb/symtab.c
@@ -4258,7 +4258,7 @@ find_epilogue_using_linetable (CORE_ADDR func_addr)
       /* Move to the last linetable entry of the current function.  */
       if (it == &linetable->item[0])
 	{
-	  /* Doing it-- would introduce undefined behaviour, avoid it by
+	  /* Doing it-- would introduce undefined behavior, avoid it by
 	     explicitly handling this case.  */
 	  return {};
 	}
@@ -4292,7 +4292,7 @@ find_epilogue_using_linetable (CORE_ADDR func_addr)
 	  if (it == &linetable->item[0])
 	    {
 	      /* No more entries in the current function.
-		 Doing it-- would introduce undefined behaviour, avoid it by
+		 Doing it-- would introduce undefined behavior, avoid it by
 		 explicitly handling this case.  */
 	      break;
 	    }
diff --git a/gdb/target.h b/gdb/target.h
index d872991..be84b27 100644
--- a/gdb/target.h
+++ b/gdb/target.h
@@ -1661,7 +1661,7 @@ struct memory_write_request
   void *baton;
 };
 
-/* Enumeration specifying different flash preservation behaviour.  */
+/* Enumeration specifying different flash preservation behavior.  */
 enum flash_preserve_mode
   {
     flash_preserve,
diff --git a/gdb/testsuite/Makefile.in b/gdb/testsuite/Makefile.in
index f799f16..3235c70 100644
--- a/gdb/testsuite/Makefile.in
+++ b/gdb/testsuite/Makefile.in
@@ -209,7 +209,7 @@ endif
 # in the same situation, we avoid invoking DejaGnu, and instead just call
 # the check/no-matching-tests-found rule (which prints a helpful message).
 #
-# To get the same behaviour for check-single we decide here, based on how
+# To get the same behavior for check-single we decide here, based on how
 # TESTS expanded, whether check-single should redirect to do-check-single or
 # to check/no-matching-tests-found.
 ifeq ($(expanded_tests_or_none),no-matching-tests-found)
diff --git a/gdb/testsuite/gdb.arch/amd64-invalid-stack-middle.c b/gdb/testsuite/gdb.arch/amd64-invalid-stack-middle.c
index 7d8a667..1e0e957 100644
--- a/gdb/testsuite/gdb.arch/amd64-invalid-stack-middle.c
+++ b/gdb/testsuite/gdb.arch/amd64-invalid-stack-middle.c
@@ -58,7 +58,7 @@ func1 (void *ptr)
 
 /* Finds and returns an invalid pointer, mmaps in a page, grabs a pointer
    to it then unmaps the page again.  This is almost certainly "undefined"
-   behaviour, but should be good enough for this small test program.  */
+   behavior, but should be good enough for this small test program.  */
 
 static void *
 make_invalid_ptr (void)
diff --git a/gdb/testsuite/gdb.arch/amd64-invalid-stack-middle.exp b/gdb/testsuite/gdb.arch/amd64-invalid-stack-middle.exp
index 2423e76..d78c051 100644
--- a/gdb/testsuite/gdb.arch/amd64-invalid-stack-middle.exp
+++ b/gdb/testsuite/gdb.arch/amd64-invalid-stack-middle.exp
@@ -19,7 +19,7 @@
 # frame where the stack in unreadable.
 #
 # One interesting bug that has been observed is that gdb will sometime
-# exhibit different behaviour the first time a stack command is run
+# exhibit different behavior the first time a stack command is run
 # compared to the second (and later) times a command is run.  This is
 # because the first time a command is run gdb actually tries to figure out
 # the answer, while the second (and later) times gdb relies on the answer
diff --git a/gdb/testsuite/gdb.arch/amd64-invalid-stack-top.c b/gdb/testsuite/gdb.arch/amd64-invalid-stack-top.c
index 8373693..445673c 100644
--- a/gdb/testsuite/gdb.arch/amd64-invalid-stack-top.c
+++ b/gdb/testsuite/gdb.arch/amd64-invalid-stack-top.c
@@ -42,7 +42,7 @@ func1 (void *ptr)
 
 /* Finds and returns an invalid pointer, mmaps in a page, grabs a pointer
    to it then unmaps the page again.  This is almost certainly "undefined"
-   behaviour, but should be good enough for this small test program.  */
+   behavior, but should be good enough for this small test program.  */
 
 static void *
 make_invalid_ptr (void)
diff --git a/gdb/testsuite/gdb.arch/amd64-invalid-stack-top.exp b/gdb/testsuite/gdb.arch/amd64-invalid-stack-top.exp
index 8571ca9..b58a75f 100644
--- a/gdb/testsuite/gdb.arch/amd64-invalid-stack-top.exp
+++ b/gdb/testsuite/gdb.arch/amd64-invalid-stack-top.exp
@@ -19,7 +19,7 @@
 # frame where the stack in unreadable.
 #
 # One interesting bug that has been observed is that gdb will sometime
-# exhibit different behaviour the first time a stack command is run
+# exhibit different behavior the first time a stack command is run
 # compared to the second (and later) times a command is run.  This is
 # because the first time a command is run gdb actually tries to figure out
 # the answer, while the second (and later) times gdb relies on the answer
diff --git a/gdb/testsuite/gdb.base/bt-on-fatal-signal.exp b/gdb/testsuite/gdb.base/bt-on-fatal-signal.exp
index 9bed64a..81a634d 100644
--- a/gdb/testsuite/gdb.base/bt-on-fatal-signal.exp
+++ b/gdb/testsuite/gdb.base/bt-on-fatal-signal.exp
@@ -13,7 +13,7 @@
 # You should have received a copy of the GNU General Public License
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
-# Test the 'maint set backtrace-on-fatal-signal' behaviour.  Start up
+# Test the 'maint set backtrace-on-fatal-signal' behavior.  Start up
 # GDB, turn on backtrace-on-fatal-signal, then send fatal signals to
 # GDB and ensure we see the backtrace.
 
diff --git a/gdb/testsuite/gdb.base/cached-source-file.exp b/gdb/testsuite/gdb.base/cached-source-file.exp
index a2df5dc..b063362 100644
--- a/gdb/testsuite/gdb.base/cached-source-file.exp
+++ b/gdb/testsuite/gdb.base/cached-source-file.exp
@@ -124,7 +124,7 @@ file rename -force -- $bkpsrc $srcfile
 # As before, delay so that at least one second has passed.  GDB still
 # will not spot that the source file has changed, as GDB doesn't do a
 # time check unless the binary has also changed, this delay just
-# allows us to confirm this behaviour.
+# allows us to confirm this behavior.
 sleep 1
 
 # List the printf line again, we should not see the file changes yet
diff --git a/gdb/testsuite/gdb.base/callexit.exp b/gdb/testsuite/gdb.base/callexit.exp
index 692d9d0..f2b0838 100644
--- a/gdb/testsuite/gdb.base/callexit.exp
+++ b/gdb/testsuite/gdb.base/callexit.exp
@@ -33,7 +33,7 @@ if { ![runto_main] } {
     return 0
 }
 
-# Verify set may-call-functions behaviour.
+# Verify set may-call-functions behavior.
 gdb_test_no_output "set may-call-functions off"
 gdb_test "call callexit()" \
     "Cannot call functions in the program: may-call-functions is off." \
diff --git a/gdb/testsuite/gdb.base/condbreak.exp b/gdb/testsuite/gdb.base/condbreak.exp
index 3b619e4..1f07bb5 100644
--- a/gdb/testsuite/gdb.base/condbreak.exp
+++ b/gdb/testsuite/gdb.base/condbreak.exp
@@ -252,7 +252,7 @@ gdb_test "with breakpoint pending off -- break main ta 999 if (1==1)" \
 # as " - force - condition", that is, subtraction of the symbol
 # "force" followed by subtraction of symbol "context".  This is really
 # just a quirk of how this used to be implemented, and should maybe be
-# changed in the future.  However, for now GDB retains this behaviour.
+# changed in the future.  However, for now GDB retains this behavior.
 gdb_test "break *main if (1==1) -force-condition" \
     "No symbol \"force\" in current context\\."
 
diff --git a/gdb/testsuite/gdb.base/define-prefix.exp b/gdb/testsuite/gdb.base/define-prefix.exp
index 5695c1a..81d3a56 100644
--- a/gdb/testsuite/gdb.base/define-prefix.exp
+++ b/gdb/testsuite/gdb.base/define-prefix.exp
@@ -152,7 +152,7 @@ gdb_test "abc-prefix def-prefix ghi-prefix-cmd alternate-jkl-cmd" \
     "use user command after redefining ghi-prefix-cmd: alternate-jkl-cmd"
 
 ####################
-# Check error behaviour.
+# Check error behavior.
 
 gdb_test "define-prefix print" ".* is built-in.*"
 
diff --git a/gdb/testsuite/gdb.base/exitsignal.exp b/gdb/testsuite/gdb.base/exitsignal.exp
index 85fd956..2d6fc03 100644
--- a/gdb/testsuite/gdb.base/exitsignal.exp
+++ b/gdb/testsuite/gdb.base/exitsignal.exp
@@ -80,7 +80,7 @@ gdb_test "print \$_exitsignal" " = 11" \
 gdb_test "print \$_exitcode" " = void" \
     "\$_exitcode is still void after restarting the inferior"
 
-# Now we test the behaviour of $_exit{code,signal} during a normal
+# Now we test the behavior of $_exit{code,signal} during a normal
 # inferior execution.
 standard_testfile normal.c
 
diff --git a/gdb/testsuite/gdb.base/until-trailing-insns.exp b/gdb/testsuite/gdb.base/until-trailing-insns.exp
index baa5f1f..ffd5df8 100644
--- a/gdb/testsuite/gdb.base/until-trailing-insns.exp
+++ b/gdb/testsuite/gdb.base/until-trailing-insns.exp
@@ -74,7 +74,7 @@
 # range.  It is only when GDB exits the loop that we leave the stepping
 # range, and the stepping finishes at address 0x114.
 #
-# This test checks this behaviour using the DWARF assembler.
+# This test checks this behavior using the DWARF assembler.
 
 load_lib dwarf.exp
 
diff --git a/gdb/testsuite/gdb.base/wrap-line.exp b/gdb/testsuite/gdb.base/wrap-line.exp
index 323cd6c..d5001bf 100644
--- a/gdb/testsuite/gdb.base/wrap-line.exp
+++ b/gdb/testsuite/gdb.base/wrap-line.exp
@@ -20,7 +20,7 @@
 require {!is_remote host}
 
 # Test both ansi (no auto-wrap) and xterm (auto-wrap).  Also test dumb, which
-# shows the effective behaviour on stub-termcap systems, regardless of the
+# shows the effective behavior on stub-termcap systems, regardless of the
 # TERM setting.
 set terms {ansi xterm dumb}
 
diff --git a/gdb/testsuite/gdb.btrace/ptwrite.exp b/gdb/testsuite/gdb.btrace/ptwrite.exp
index 0970d31..7f1adad 100644
--- a/gdb/testsuite/gdb.btrace/ptwrite.exp
+++ b/gdb/testsuite/gdb.btrace/ptwrite.exp
@@ -145,7 +145,7 @@ with_test_prefix "Custom" {
 	]
 }
 
-### 2.2 None as filter. This resets the default behaviour.
+### 2.2 None as filter. This resets the default behavior.
 with_test_prefix "None" {
     gdb_test_multiline "register filter in python" \
 	"python" "" \
diff --git a/gdb/testsuite/gdb.cp/empty-enum.exp b/gdb/testsuite/gdb.cp/empty-enum.exp
index 6acae55..6816e86 100644
--- a/gdb/testsuite/gdb.cp/empty-enum.exp
+++ b/gdb/testsuite/gdb.cp/empty-enum.exp
@@ -25,7 +25,7 @@
 #   (gdb) print enum_var
 #   $1 = 8
 #
-# This test checks this behaviour.
+# This test checks this behavior.
 
 standard_testfile .cc
 
diff --git a/gdb/testsuite/gdb.cp/gdb2495.exp b/gdb/testsuite/gdb.cp/gdb2495.exp
index 7fbadb2..86ee7f9 100644
--- a/gdb/testsuite/gdb.cp/gdb2495.exp
+++ b/gdb/testsuite/gdb.cp/gdb2495.exp
@@ -24,7 +24,7 @@
 # in-frame).  The default handler calls std::terminate.  This will kill
 # the inferior.  Assert that terminate should never be called in an
 # inferior function call.  These tests test the functionality around
-# unwinding that sequence and also tests the flag behaviour gating this
+# unwinding that sequence and also tests the flag behavior gating this
 # functionality.
 #
 # PR c++/9600.
@@ -51,7 +51,7 @@ if {![runto_main]} {
 
 # See http://sourceware.org/gdb/bugs/2495
 
-# Test normal baseline behaviour. Call a function that
+# Test normal baseline behavior. Call a function that
 # does not raise an exception.
 gdb_test "p exceptions.no_throw_function()" " = 1"
 # And one that does but handles it in-frame.
@@ -75,7 +75,7 @@ gdb_test "bt" \
 gdb_test "info breakpoints" \
     "gdb2495\.cc.*"
 
-# Turn off this new behaviour.
+# Turn off this new behavior.
 gdb_test_no_output "set unwind-on-terminating-exception off"
 
 # Check that it is turned off.
@@ -83,7 +83,7 @@ gdb_test "show unwind-on-terminating-exception" \
     "exception is unhandled while in a call dummy is off.*" \
     "turn off unwind on terminating exception flag"
 
-# Check that the old behaviour is restored.
+# Check that the old behavior is restored.
 gdb_test "p exceptions.throw_function()" \
     "The program being debugged was signaled while in a function called .*" \
     "call a function that raises an exception with unwinding off.."
@@ -94,10 +94,10 @@ if {![runto_main]} {
 }
 
 
-# Check to see if the new behaviour alters the unwind signal
+# Check to see if the new behavior alters the unwind signal
 # behaviour; it should not.  Test both on and off states.
 
-# Turn on unwind on signal behaviour.
+# Turn on unwind on signal behavior.
 gdb_test_no_output "set unwind-on-signal on"
 
 # Check that it is turned on.
@@ -105,7 +105,7 @@ gdb_test "show unwind-on-signal" \
     "signal is received while in a call dummy is on.*" \
     "turn on unwind on signal"
 
-# Check to see if new behaviour interferes with
+# Check to see if new behavior interferes with
 # normal signal handling in inferior function calls.
 gdb_test "p exceptions.raise_signal(1)" \
     [multi_line \
@@ -124,7 +124,7 @@ gdb_test "show unwind-on-signal" \
     "signal is received while in a call dummy is off.*" \
     "turn off unwind on signal"
 
-# Check to see if new behaviour interferes with
+# Check to see if new behavior interferes with
 # normal signal handling in inferior function calls.
 gdb_test "p exceptions.raise_signal(1)" \
     "To change this behavior use \"set unwind-on-signal on\".*" \
diff --git a/gdb/testsuite/gdb.debuginfod/corefile-mapped-file-3.c b/gdb/testsuite/gdb.debuginfod/corefile-mapped-file-3.c
index 98ed952..e060866 100644
--- a/gdb/testsuite/gdb.debuginfod/corefile-mapped-file-3.c
+++ b/gdb/testsuite/gdb.debuginfod/corefile-mapped-file-3.c
@@ -38,7 +38,7 @@ main ()
   library_base_address
     = mmap (NULL, buf.st_size, PROT_READ, MAP_PRIVATE, fd, 0);
 
-  res = *ptr;	/* Undefined behaviour here.  */
+  res = *ptr;	/* Undefined behavior here.  */
 
   return 0;
 }
diff --git a/gdb/testsuite/gdb.debuginfod/corefile-mapped-file.exp b/gdb/testsuite/gdb.debuginfod/corefile-mapped-file.exp
index cad70aa..f02876e 100644
--- a/gdb/testsuite/gdb.debuginfod/corefile-mapped-file.exp
+++ b/gdb/testsuite/gdb.debuginfod/corefile-mapped-file.exp
@@ -169,7 +169,7 @@ if {![runto_main]} {
     return
 }
 
-gdb_breakpoint [gdb_get_line_number "Undefined behaviour here" $srcfile3]
+gdb_breakpoint [gdb_get_line_number "Undefined behavior here" $srcfile3]
 gdb_continue_to_breakpoint "run to breakpoint"
 
 set library_base_address \
diff --git a/gdb/testsuite/gdb.dlang/expression.exp b/gdb/testsuite/gdb.dlang/expression.exp
index 8f72ec0..c3b8bd2 100644
--- a/gdb/testsuite/gdb.dlang/expression.exp
+++ b/gdb/testsuite/gdb.dlang/expression.exp
@@ -107,7 +107,7 @@ proc test_d_float_literals {} {
 }
 
 proc test_d_expressions {} {
-    # Test expression behaviour specific to D.
+    # Test expression behavior specific to D.
 
     # Comparison and order expressions have same precedence.
     gdb_test "print 1 == 2 > 0" "A syntax error in expression, near `> 0'\."
diff --git a/gdb/testsuite/gdb.dwarf2/dw2-inline-header-2.exp b/gdb/testsuite/gdb.dwarf2/dw2-inline-header-2.exp
index 68f1fc8..4fd4c5a 100644
--- a/gdb/testsuite/gdb.dwarf2/dw2-inline-header-2.exp
+++ b/gdb/testsuite/gdb.dwarf2/dw2-inline-header-2.exp
@@ -34,7 +34,7 @@
 # third breakpoint will actually be placed at the same location as the
 # second breakpoint.
 #
-# This test is designed to test GDB's internal behaviour with respect
+# This test is designed to test GDB's internal behavior with respect
 # to discarding particular line table entries.  GCC and DWARF are
 # starting to introduce the idea of line table views.  As the views
 # information becomes better supported within GDB it is likely that
diff --git a/gdb/testsuite/gdb.dwarf2/dw2-inline-stepping.c b/gdb/testsuite/gdb.dwarf2/dw2-inline-stepping.c
index 6b0e7ac..109a372 100644
--- a/gdb/testsuite/gdb.dwarf2/dw2-inline-stepping.c
+++ b/gdb/testsuite/gdb.dwarf2/dw2-inline-stepping.c
@@ -14,7 +14,7 @@
    along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
 
 /* This test relies on foo being inlined into main and bar not being
-   inlined.  The test is checking GDB's behaviour as we single step from
+   inlined.  The test is checking GDB's behavior as we single step from
    main through foo and into bar.  */
 
 volatile int global_var;
diff --git a/gdb/testsuite/gdb.dwarf2/dw2-missing-cu-tag.exp b/gdb/testsuite/gdb.dwarf2/dw2-missing-cu-tag.exp
index f84142a..6206fb3 100644
--- a/gdb/testsuite/gdb.dwarf2/dw2-missing-cu-tag.exp
+++ b/gdb/testsuite/gdb.dwarf2/dw2-missing-cu-tag.exp
@@ -52,7 +52,7 @@ clean_restart
 gdb_test_no_output "maint set dwarf synchronous on"
 
 # This pattern is hit when GDB does not use -readnow (i.e. the default
-# behaviour).
+# behavior).
 set pattern1 \
     [multi_line \
 	 "Reading symbols from \[^\r\n\]+" \
diff --git a/gdb/testsuite/gdb.dwarf2/dw2-op-out-param.S b/gdb/testsuite/gdb.dwarf2/dw2-op-out-param.S
index 0be29ff..a271dbc 100644
--- a/gdb/testsuite/gdb.dwarf2/dw2-op-out-param.S
+++ b/gdb/testsuite/gdb.dwarf2/dw2-op-out-param.S
@@ -16,7 +16,7 @@
    along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
 
 
-/* Test the behaviour of gdb in the following situation, the dwarf debug
+/* Test the behavior of gdb in the following situation, the dwarf debug
    information describes a parameter as being in a register but a more
    recent (inner) frame marks the register as being undefined.
 
@@ -46,7 +46,7 @@
    clobbering them.
 
    The dwarf register numbering is different to the gdb register number.
-   In some of the tests we rely on gdb behaviour of being able to specify a
+   In some of the tests we rely on gdb behavior of being able to specify a
    struct using a single register location, the structure will then "flow"
    into the next gdb register.  The initial register is specified using a
    dwarf register number, but the "next" register will depend on gdb
diff --git a/gdb/testsuite/gdb.dwarf2/dw2-using-debug-str.exp b/gdb/testsuite/gdb.dwarf2/dw2-using-debug-str.exp
index c84a530..762ff02 100644
--- a/gdb/testsuite/gdb.dwarf2/dw2-using-debug-str.exp
+++ b/gdb/testsuite/gdb.dwarf2/dw2-using-debug-str.exp
@@ -134,7 +134,7 @@ set line1 "Reading symbols from \[^\r\n\]+"
 set dwarf_error "DWARF Error: DW_FORM_strp used without required section"
 
 # This pattern is hit when GDB does not use -readnow (i.e. the default
-# behaviour).
+# behavior).
 set pattern1 \
     [multi_line \
          $line1 \
diff --git a/gdb/testsuite/gdb.fortran/array-slices.exp b/gdb/testsuite/gdb.fortran/array-slices.exp
index 919a2c6..1fa6a5d 100644
--- a/gdb/testsuite/gdb.fortran/array-slices.exp
+++ b/gdb/testsuite/gdb.fortran/array-slices.exp
@@ -245,7 +245,7 @@ proc run_test { repack } {
 	    # The goal here is to compare the sizes of the slice in show with
 	    # the size of the slice extracted by GDB.  So we can only compare
 	    # sizes when GDB's repacking setting matches the repacking
-	    # behaviour we got from the compiler.
+	    # behavior we got from the compiler.
 	    if { ($addr_in_show < $start_addr || $addr_in_show >= $end_addr) \
 		 == ($repack == "on") } {
 		gdb_assert {$size_in_show == $size_in_parent} \
diff --git a/gdb/testsuite/gdb.fortran/nested-funcs-2.exp b/gdb/testsuite/gdb.fortran/nested-funcs-2.exp
index 6af9cd6..72d87bc 100644
--- a/gdb/testsuite/gdb.fortran/nested-funcs-2.exp
+++ b/gdb/testsuite/gdb.fortran/nested-funcs-2.exp
@@ -90,7 +90,7 @@ proc do_bp_tests {with_src_prefix_p with_nest_prefix_p} {
 		     # Currently referencing symbols using 'info',
 		     # 'whatis' and 'ptype' before the program is
 		     # started doesn't work.  This is the same
-		     # behaviour we see in C++ so I don't think this
+		     # behavior we see in C++ so I don't think this
 		     # is a failure, just a limitation in current GDB.
 		     if { ${with_nest_prefix_p} } {
 			 gdb_test "info symbol ${nest_prefix}${function}" \
diff --git a/gdb/testsuite/gdb.mi/mi-condbreak-throw.exp b/gdb/testsuite/gdb.mi/mi-condbreak-throw.exp
index e35ad8f..2d2392d 100644
--- a/gdb/testsuite/gdb.mi/mi-condbreak-throw.exp
+++ b/gdb/testsuite/gdb.mi/mi-condbreak-throw.exp
@@ -75,7 +75,7 @@ proc run_test { unwind_on_exception } {
 		 "&\"caused by an unhandled C\\+\\+ exception.  GDB blocked this call in order\\\\n\"" \
 		 "&\"to prevent the program from being terminated, and has restored the\\\\n\"" \
 		 "&\"context to its original state before the call.\\\\n\"" \
-		 "&\"To change this behaviour use \\\\\"set unwind-on-terminating-exception off\\\\\"\\.\\\\n\"" \
+		 "&\"To change this behavior use \\\\\"set unwind-on-terminating-exception off\\\\\"\\.\\\\n\"" \
 		 "&\"Evaluation of the expression containing the function \\(cond_throw\\(\\)\\)\\\\n\"" \
 		 "&\"will be abandoned.\\\\n\"" \
 		 "=breakpoint-modified,bkpt={number=\"$bpnum\",type=\"breakpoint\",\[^\r\n\]+times=\"1\",\[^\r\n\]+}" \
diff --git a/gdb/testsuite/gdb.mi/mi-vla-fortran.exp b/gdb/testsuite/gdb.mi/mi-vla-fortran.exp
index 6097c02..d0b4a22 100644
--- a/gdb/testsuite/gdb.mi/mi-vla-fortran.exp
+++ b/gdb/testsuite/gdb.mi/mi-vla-fortran.exp
@@ -158,7 +158,7 @@ gdb_expect {
 	    "$real" "get children of pvla2_not_associated"
     }
     -re "580\\^error,msg=\"value contents too large \\(\[0-9\]+ bytes\\).*${mi_gdb_prompt}$" {
-	# Undefined behaviour in gfortran.
+	# Undefined behavior in gfortran.
 	xfail $test
     }
     -re "${mi_gdb_prompt}$" {
diff --git a/gdb/testsuite/gdb.server/non-existing-program.exp b/gdb/testsuite/gdb.server/non-existing-program.exp
index 2d08e50..71b29c2 100644
--- a/gdb/testsuite/gdb.server/non-existing-program.exp
+++ b/gdb/testsuite/gdb.server/non-existing-program.exp
@@ -45,7 +45,7 @@ expect {
 	exp_continue
     }
     # Likewise, but with startup-with-shell enabled, which is the
-    # default behaviour.
+    # default behavior.
     -re "stdin/stdout redirected.*exec: non-existing-program: not found\r\nDuring startup program exited with code 127\.\r\nExiting\r\n" {
 	set saw_exiting 1
 	exp_continue
diff --git a/gdb/testsuite/gdb.server/target-exec-file.exp b/gdb/testsuite/gdb.server/target-exec-file.exp
index 91e4c5c..9a24b7a 100644
--- a/gdb/testsuite/gdb.server/target-exec-file.exp
+++ b/gdb/testsuite/gdb.server/target-exec-file.exp
@@ -189,7 +189,7 @@ proc start_inferior { testname expect_reread } {
 	}
 
 	gdb_assert { $expect_reread == $saw_reread } \
-	    "check symbol re-read behaviour"
+	    "check symbol re-read behavior"
     }
 }
 
diff --git a/gdb/testsuite/gdb.threads/slow-waitpid.c b/gdb/testsuite/gdb.threads/slow-waitpid.c
index 25052c3..280b109 100644
--- a/gdb/testsuite/gdb.threads/slow-waitpid.c
+++ b/gdb/testsuite/gdb.threads/slow-waitpid.c
@@ -19,7 +19,7 @@
    using the LD_PRELOAD technique.
 
    The library intercepts calls to WAITPID and SIGSUSPEND in order to
-   simulate the behaviour of a heavily loaded kernel.
+   simulate the behavior of a heavily loaded kernel.
 
    When GDB wants to stop all threads in an inferior each thread is sent a
    SIGSTOP, GDB will then wait for the signal to be received by the thread
@@ -39,7 +39,7 @@
    The idea in this library is to rate limit calls to waitpid (where pid is
    -1 and the WNOHANG option is set) so that only 1 per second can return
    an answer.  Any additional calls will report that no threads are
-   currently ready.  This should match the behaviour we see on a slow
+   currently ready.  This should match the behavior we see on a slow
    kernel.
 
    However, given that usually when using this library, the kernel does
diff --git a/gdb/testsuite/lib/read1.c b/gdb/testsuite/lib/read1.c
index e449717..29570c9 100644
--- a/gdb/testsuite/lib/read1.c
+++ b/gdb/testsuite/lib/read1.c
@@ -62,7 +62,7 @@ init_readmore (int *method, unsigned int *sleep, FILE **log)
     *log = fopen (env, "w");
 }
 
-/* Wrap 'read', and modify it's behaviour using READ1 or READMORE style.  */
+/* Wrap 'read', and modify it's behavior using READ1 or READMORE style.  */
 
 ssize_t
 read (int fd, void *buf, size_t count)
@@ -89,7 +89,7 @@ read (int fd, void *buf, size_t count)
 	init_readmore (&readmore_method, &readmore_sleep, &log);
     }
 
-  /* Only modify 'read' behaviour when reading from the terminal.  */
+  /* Only modify 'read' behavior when reading from the terminal.  */
   if (isatty (fd) == 0)
     goto fallback;
 
diff --git a/gdb/top.h b/gdb/top.h
index b57cc3b..07cdc5a 100644
--- a/gdb/top.h
+++ b/gdb/top.h
@@ -49,7 +49,7 @@ extern void execute_command (const char *, int);
 
 /* Run FN.  Capture its output into the returned string, do not display it
    to the screen.  The global BATCH_FLAG will temporarily be set to true.
-   When TERM_OUT is true the output is collected with terminal behaviour
+   When TERM_OUT is true the output is collected with terminal behavior
    (e.g. with styling).  When TERM_OUT is false raw output will be collected
    (e.g. no styling).  */
 
diff --git a/gdb/tui/tui-regs.c b/gdb/tui/tui-regs.c
index 38ddd23..927b11f 100644
--- a/gdb/tui/tui-regs.c
+++ b/gdb/tui/tui-regs.c
@@ -437,7 +437,7 @@ tui_register_info::rerender (WINDOW *handle, int field_width)
 
 /* Helper for "tui reg next", returns the next register group after
    CURRENT_GROUP in the register group list for GDBARCH, with wrap around
-   behaviour.
+   behavior.
 
    If CURRENT_GROUP is nullptr (e.g. if the tui register window has only
    just been displayed and has no current group selected) or the currently
@@ -459,7 +459,7 @@ tui_reg_next (const reggroup *current_group, struct gdbarch *gdbarch)
 
 /* Helper for "tui reg prev", returns the register group previous to
    CURRENT_GROUP in the register group list for GDBARCH, with wrap around
-   behaviour.
+   behavior.
 
    If CURRENT_GROUP is nullptr (e.g. if the tui register window has only
    just been displayed and has no current group selected) or the currently
diff --git a/gdb/ui-file.h b/gdb/ui-file.h
index 9686440..dcbc162 100644
--- a/gdb/ui-file.h
+++ b/gdb/ui-file.h
@@ -82,10 +82,10 @@ public:
   virtual bool isatty ()
   { return false; }
 
-  /* true indicates terminal output behaviour such as cli_styling.
+  /* true indicates terminal output behavior such as cli_styling.
      This default implementation indicates to do terminal output
-     behaviour if the UI_FILE is a tty.  A derived class can override
-     TERM_OUT to have cli_styling behaviour without being a tty.  */
+     behavior if the UI_FILE is a tty.  A derived class can override
+     TERM_OUT to have cli_styling behavior without being a tty.  */
   virtual bool term_out ()
   { return isatty (); }
 
@@ -171,9 +171,9 @@ class string_file : public ui_file
 {
 public:
   /* Construct a string_file to collect 'raw' output, i.e. without
-     'terminal' behaviour such as cli_styling.  */
+     'terminal' behavior such as cli_styling.  */
   string_file () : m_term_out (false) {};
-  /* If TERM_OUT, construct a string_file with terminal output behaviour
+  /* If TERM_OUT, construct a string_file with terminal output behavior
      such as cli_styling)
      else collect 'raw' output like the previous constructor.  */
   explicit string_file (bool term_out) : m_term_out (term_out) {};
diff --git a/gdb/varobj.c b/gdb/varobj.c
index 04f1fc8..dcadce1 100644
--- a/gdb/varobj.c
+++ b/gdb/varobj.c
@@ -1183,7 +1183,7 @@ install_new_value (struct varobj *var, struct value *value, bool initial)
   changeable = varobj_value_is_changeable_p (var);
 
   /* If the type has custom visualizer, we consider it to be always
-     changeable.  FIXME: need to make sure this behaviour will not
+     changeable.  FIXME: need to make sure this behavior will not
      mess up read-sensitive values.  */
   if (var->dynamic->pretty_printer != NULL)
     changeable = true;