]> Cypherpunks repositories - gostls13.git/commit
cmd/compile: move dumpFileSeq
authorsurechen <surechen17@gmail.com>
Mon, 27 Jul 2020 04:00:19 +0000 (12:00 +0800)
committerKeith Randall <khr@golang.org>
Mon, 17 Aug 2020 21:04:19 +0000 (21:04 +0000)
commit17553c6e7154deab8501595f4fa387c0b718337a
treec7050656ab5e4ecdd6760e9654d4320ae2ab89c4
parent99d6e3eec2cad4387b28759322273d774cc94fe5
cmd/compile: move dumpFileSeq

I noticed that there is a Todo comment here. This variable is only used for filename when dump a function's ssa passes result in details. It is no problem to print a function alone, but may be edited by not only one goroutine if dump multiple functions at the same time. Although it looks only dump one function's ssa passes now. As far as I am concerned this variable can be a member variable of the struct Func. I'm not sure if this change is necessary. Looking forward to your advices, thank you very much.

Change-Id: I35dd7247889e0cc7f19c0b400b597206592dee75
Reviewed-on: https://go-review.googlesource.com/c/go/+/244918
Reviewed-by: Keith Randall <khr@golang.org>
Run-TryBot: Keith Randall <khr@golang.org>
src/cmd/compile/internal/ssa/compile.go
src/cmd/compile/internal/ssa/func.go