Commentary: Spelling (and grammar) fixes (#3846)

ci/ci-script.bash

@@ -69,7 +69,7 @@ elif [ "$CI_BUILD_STAGE_NAME" = "test" ]; then
 
   if [ "$CI_OS_NAME" = "osx" ]; then
     export VERILATOR_TEST_NO_GDB=1  # Pain to get GDB to work on OS X
-    # TODO below may no longer be requried as configure checks for -pg
+    # TODO below may no longer be required as configure checks for -pg
     export VERILATOR_TEST_NO_GPROF=1  # Apple Clang has no -pg
     # export PATH="/Applications/gtkwave.app/Contents/Resources/bin:$PATH" # fst2vcd
     file bin/verilator_bin
@@ -85,7 +85,7 @@ elif [ "$CI_BUILD_STAGE_NAME" = "test" ]; then
     "$MAKE" -j "$NPROC" -k
   elif [ "$CI_OS_NAME" = "freebsd" ]; then
     export VERILATOR_TEST_NO_GDB=1 # Disable for now, ideally should run
-    # TODO below may no longer be requried as configure checks for -pg
+    # TODO below may no longer be required as configure checks for -pg
     export VERILATOR_TEST_NO_GPROF=1 # gprof is a bit different on FreeBSD, disable
   fi
 

configure.ac

@@ -112,7 +112,7 @@ AC_MSG_CHECKING(whether to show and stop on compilation warnings)
 AC_ARG_ENABLE([ccwarn],
               [AS_HELP_STRING([--enable-ccwarn],
                               [enable showing and stopping on compilation warnings
-                               in Verilator binrary and Verilated makefiles])],
+                               in Verilator binary and Verilated makefiles])],
               [case "${enableval}" in
                 yes) CFG_WITH_CCWARN=yes ;;
                 no)  CFG_WITH_CCWARN=no ;;
@@ -351,7 +351,7 @@ AC_SUBST(CFG_CXXFLAGS_PROFILE)
 # Currently enable c++17/c++14 due to packaged SystemC dependency
 # c++17 is the newest that Verilator is regularly tested to support
 # c++11 is the oldest that Verilator supports
-# gnu is requried for Cygwin to compile verilated.h successfully
+# gnu is required for Cygwin to compile verilated.h successfully
 #_MY_CXX_CHECK_SET(CFG_CXXFLAGS_STD_NEWEST,-std=gnu++20)
 #_MY_CXX_CHECK_SET(CFG_CXXFLAGS_STD_NEWEST,-std=c++20)
 case "$(which lsb_release 2>&1 > /dev/null && lsb_release -d)" in

nodist/fuzzer/all

@@ -10,7 +10,7 @@
 ######################################################################
 
 # Run all steps needed to configure and start fuzzer
-# Note that this assumes the system is a Debian-like Linux distrubution
+# Note that this assumes the system is a Debian-like Linux distribution
 
 set -e
 

test_regress/t/t_bitsel_wire_array_bad.pl

@@ -10,7 +10,7 @@ if (!$::Driver) { use FindBin; exec("$FindBin::Bin/bootstrap.pl", @ARGV, $0); di
 
 scenarios(linter => 1);
 
-# Comple time only test
+# Compile time only test
 
 lint(
     fails => 1,

test_regress/t/t_dist_tabs.pl

@@ -94,7 +94,7 @@ sub _has_tabs {
     if ($filename =~ /\.out$/) {
         # Ignore golden files
     } elsif ($contents =~ /[\001\002\003\004\005\006]/) {
-        # Ignore binrary files
+        # Ignore binary files
     } elsif ($contents =~ /\t/) {
         return 1;
     }

test_regress/t/t_dist_untracked.pl

@@ -49,7 +49,7 @@ sub _has_tabs {
     if ($filename =~ /\.out$/) {
         # Ignore golden files
     } elsif ($contents =~ /[\001\002\003\004\005\006]/) {
-        # Ignore binrary files
+        # Ignore binary files
     } elsif ($contents =~ /\t/) {
         return 1;
     }

test_regress/t/t_flag_csplit.pl

@@ -11,7 +11,7 @@ if (!$::Driver) { use FindBin; exec("$FindBin::Bin/bootstrap.pl", @ARGV, $0); di
 scenarios(vlt_all => 1);
 
 while (1) {
-    # Thi rule requires GNU make > 4.1 (or so, known broken in 3.81)
+    # This rule requires GNU make > 4.1 (or so, known broken in 3.81)
     #%__Slow.o: %__Slow.cpp
     #        $(OBJCACHE) $(CXX) $(CXXFLAGS) $(CPPFLAGS) $(OPT_SLOW) -c -o $@ $<
     if (make_version() < 4.1) {

test_regress/t/t_flag_csplit_off.pl

@@ -13,7 +13,7 @@ scenarios(vlt_all => 1);
 top_filename("t/t_flag_csplit.v");
 
 while (1) {
-    # Thi rule requires GNU make > 4.1 (or so, known broken in 3.81)
+    # This rule requires GNU make > 4.1 (or so, known broken in 3.81)
     #%__Slow.o: %__Slow.cpp
     #        $(OBJCACHE) $(CXX) $(CXXFLAGS) $(CPPFLAGS) $(OPT_SLOW) -c -o $@ $<
     if (make_version() < 4.1) {

test_regress/t/t_flag_parameter_hier.pl

@@ -11,7 +11,7 @@ if (!$::Driver) { use FindBin; exec("$FindBin::Bin/bootstrap.pl", @ARGV, $0); di
 scenarios(simulator => 1);
 
 compile(
-    # For Verilator, all PARAMs at all levels are overwridden
+    # For Verilator, all PARAMs at all levels are overridden
     # Error if parameter not found
     #verilator_flags2 => ['-GPARAM=10 -Gtop.t.x.HIER=20'],  # HIER would error
     verilator_flags2 => ['-GPARAM=10'],

test_regress/t/t_hier_block.pl

@@ -14,7 +14,7 @@ scenarios(vlt_all => 1);
 
 # CI environment offers 2 VCPUs, 2 thread setting causes the following warning.
 # %Warning-UNOPTTHREADS: Thread scheduler is unable to provide requested parallelism; consider asking for fewer threads.
-# So use 6 threads here though it's not optimal in performace wise, but ok.
+# So use 6 threads here though it's not optimal in performance, but ok.
 
 compile(
     v_flags2 => ['t/t_hier_block.cpp'],

test_regress/t/t_hier_block_cmake.pl

@@ -7,7 +7,7 @@ if (!$::Driver) { use FindBin; exec("$FindBin::Bin/bootstrap.pl", @ARGV, $0); di
 # Lesser General Public License Version 3 or the Perl Artistic License
 # Version 2.0.
 
-# If a test fails, broken .cmake may distrub the next run
+# If a test fails, broken .cmake may disturb the next run
 clean_objs();
 
 scenarios(simulator => 1);

test_regress/t/t_hier_block_nohier.pl

@@ -7,7 +7,7 @@ if (!$::Driver) { use FindBin; exec("$FindBin::Bin/bootstrap.pl", @ARGV, $0); di
 # Lesser General Public License Version 3 or the Perl Artistic License
 # Version 2.0.
 
-# This test makes sure that the intenral check of t_hier_block.v is correct.
+# This test makes sure that the internal check of t_hier_block.v is correct.
 # --hierarchical option is not set intentionally.
 
 # stats will be deleted but generation will be skipped if libs of hierarchical blocks exist.
@@ -18,7 +18,7 @@ top_filename("t/t_hier_block.v");
 
 # CI environment offers 2 VCPUs, 2 thread setting causes the following warning.
 # %Warning-UNOPTTHREADS: Thread scheduler is unable to provide requested parallelism; consider asking for fewer threads.
-# So use 6 threads here though it's not optimal in performace wise, but ok.
+# So use 6 threads here though it's not optimal in performance, but ok.
 compile(
     v_flags2 => ['t/t_hier_block.cpp'],
     verilator_flags2 => ['--stats',

test_regress/t/t_hier_block_sc.pl

@@ -14,7 +14,7 @@ top_filename("t/t_hier_block.v");
 
 # CI environment offers 2 VCPUs, 2 thread setting causes the following warning.
 # %Warning-UNOPTTHREADS: Thread scheduler is unable to provide requested parallelism; consider asking for fewer threads.
-# So use 6 threads here though it's not optimal in performace wise, but ok.
+# So use 6 threads here though it's not optimal in performance, but ok.
 scenarios(vlt_all => 1);
 
 compile(

test_regress/t/t_hier_block_sc_trace_fst.pl

@@ -14,7 +14,7 @@ top_filename("t/t_hier_block.v");
 
 # CI environment offers 2 VCPUs, 2 thread setting causes the following warning.
 # %Warning-UNOPTTHREADS: Thread scheduler is unable to provide requested parallelism; consider asking for fewer threads.
-# So use 6 threads here though it's not optimal in performace wise, but ok.
+# So use 6 threads here though it's not optimal in performance, but ok.
 scenarios(vlt_all => 1);
 
 compile(

test_regress/t/t_hier_block_sc_trace_vcd.pl

@@ -14,7 +14,7 @@ top_filename("t/t_hier_block.v");
 
 # CI environment offers 2 VCPUs, 2 thread setting causes the following warning.
 # %Warning-UNOPTTHREADS: Thread scheduler is unable to provide requested parallelism; consider asking for fewer threads.
-# So use 6 threads here though it's not optimal in performace wise, but ok.
+# So use 6 threads here though it's not optimal in performance, but ok.
 scenarios(vlt_all => 1);
 
 compile(

test_regress/t/t_hier_block_trace_fst.pl

@@ -13,7 +13,7 @@ top_filename("t/t_hier_block.v");
 
 # CI environment offers 2 VCPUs, 2 thread setting causes the following warning.
 # %Warning-UNOPTTHREADS: Thread scheduler is unable to provide requested parallelism; consider asking for fewer threads.
-# So use 6 threads here though it's not optimal in performace wise, but ok.
+# So use 6 threads here though it's not optimal in performance, but ok.
 
 compile(
     v_flags2 => ['t/t_hier_block.cpp'],

test_regress/t/t_hier_block_trace_vcd.pl

@@ -13,7 +13,7 @@ top_filename("t/t_hier_block.v");
 
 # CI environment offers 2 VCPUs, 2 thread setting causes the following warning.
 # %Warning-UNOPTTHREADS: Thread scheduler is unable to provide requested parallelism; consider asking for fewer threads.
-# So use 6 threads here though it's not optimal in performace wise, but ok.
+# So use 6 threads here though it's not optimal in performance, but ok.
 
 compile(
     v_flags2 => ['t/t_hier_block.cpp'],

test_regress/t/t_hier_block_vlt.pl

@@ -15,7 +15,7 @@ top_filename("t/t_hier_block.v");
 
 # CI environment offers 2 VCPUs, 2 thread setting causes the following warning.
 # %Warning-UNOPTTHREADS: Thread scheduler is unable to provide requested parallelism; consider asking for fewer threads.
-# So use 6 threads here though it's not optimal in performace wise, but ok.
+# So use 6 threads here though it's not optimal in performance, but ok.
 compile(
     v_flags2 => ['t/t_hier_block.cpp'],
     verilator_flags2 => ['--stats',

test_regress/t/t_lint_literal_bad.pl

@@ -3,7 +3,7 @@ if (!$::Driver) { use FindBin; exec("$FindBin::Bin/bootstrap.pl", @ARGV, $0); di
 # DESCRIPTION: Verilator: Verilog Test driver/expect definition
 #
 # Copyright 2017 by Todd Strader. This program is free software; you
-# can predistribute it and/or modify it under the terms of either the GNU
+# can redistribute it and/or modify it under the terms of either the GNU
 # Lesser General Public License Version 3 or the Perl Artistic License
 # Version 2.0.
 # SPDX-License-Identifier: LGPL-3.0-only OR Artistic-2.0

test_regress/t/t_pgo_threads.pl

@@ -29,7 +29,7 @@ execute(
 file_grep("$Self->{obj_dir}/profile.vlt", qr/profile_data/i);
 
 compile(
-    # Intentinally no --prof-pgo here to make sure profile data can be read in
+    # Intentionally no --prof-pgo here to make sure profile data can be read in
     # without it (that is: --prof-pgo has no effect on profile_data hash names)
     v_flags2 => [" $Self->{obj_dir}/profile.vlt"],
     threads => 2

test_regress/t/t_prof.pl

@@ -10,7 +10,7 @@ if (!$::Driver) { use FindBin; exec("$FindBin::Bin/bootstrap.pl", @ARGV, $0); di
 
 scenarios(vlt_all => 1);
 
-# TODO below might no longer be requried as configure checks for -pg
+# TODO below might no longer be required as configure checks for -pg
 if ($ENV{VERILATOR_TEST_NO_GPROF}) {
     skip("Skipping due to VERILATOR_TEST_NO_GPROF");
 } else {

test_regress/t/t_split_var_0.pl

@@ -12,7 +12,7 @@ scenarios(simulator => 1);
 
 # CI environment offers 2 VCPUs, 2 thread setting causes the following warning.
 # %Warning-UNOPTTHREADS: Thread scheduler is unable to provide requested parallelism; consider asking for fewer threads.
-# So use 6 threads here though it's not optimal in performace wise, but ok.
+# So use 6 threads here though it's not optimal in performance, but ok.
 compile(
     verilator_flags2 => ['--stats', "$Self->{t_dir}/t_split_var_0.vlt"],
     threads => $Self->{vltmt} ? 6 : 1

test_regress/t/t_split_var_2_trace.pl

@@ -13,7 +13,7 @@ top_filename("t/t_split_var_0.v");
 
 # CI environment offers 2 VCPUs, 2 thread setting causes the following warning.
 # %Warning-UNOPTTHREADS: Thread scheduler is unable to provide requested parallelism; consider asking for fewer threads.
-# So use 6 threads here though it's not optimal in performace wise, but ok.
+# So use 6 threads here though it's not optimal in performance, but ok.
 compile(
     verilator_flags2 => ['--cc --trace --stats +define+TEST_ATTRIBUTES'],
     threads => $Self->{vltmt} ? 6 : 1

test_regress/t/t_tri_ifbegin.pl

@@ -13,7 +13,7 @@ scenarios(simulator => 1);
 compile(
     );
 
-# No exeecution
+# No execution
 
 ok(1);
 1;