cmd/compile/internal/syntax: use strings.LastIndexByte in trailingDigits

Previously, strings.LastIndexByte couldn't be used because it was only
added in Go 1.5 but Go 1.4 was required for bootstrapping. In Go 1.18,
the bootstrap toolchain was bumped to Go 1.17 (see #44505), thus
strings.LastIndexByte can be used now.

Change-Id: I01a70a59dbfc853cf03d49747a2dd62d21ba74e9
Reviewed-on: https://go-review.googlesource.com/c/go/+/522197
Reviewed-by: Ian Lance Taylor <iant@google.com>
Run-TryBot: Tobias Klauser <tobias.klauser@gmail.com>
Reviewed-by: Matthew Dempsky <mdempsky@google.com>
Auto-Submit: Tobias Klauser <tobias.klauser@gmail.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Daniel Martí <mvdan@mvdan.cc>

diff --git a/src/cmd/compile/internal/syntax/parser.go b/src/cmd/compile/internal/syntax/parser.go
index 7085287cad04d69619b448b6db75684121d5fe61..913a2f164cb64b39cb5597d73d51348c33518887 100644 (file)
--- a/src/cmd/compile/internal/syntax/parser.go
+++ b/src/cmd/compile/internal/syntax/parser.go
@@ -181,10 +181,9 @@ func commentText(s string) string {
 }
 
 func trailingDigits(text string) (uint, uint, bool) {
-       // Want to use LastIndexByte below but it's not defined in Go1.4 and bootstrap fails.
-       i := strings.LastIndex(text, ":") // look from right (Windows filenames may contain ':')
+       i := strings.LastIndexByte(text, ':') // look from right (Windows filenames may contain ':')
        if i < 0 {
-               return 0, 0, false // no ":"
+               return 0, 0, false // no ':'
        }
        // i >= 0
        n, err := strconv.ParseUint(text[i+1:], 10, 0)