Commit 05c35ac5 authored by Marvin Stenger's avatar Marvin Stenger Committed by Brad Fitzpatrick

cmd/compile/internal/gc: convert fields of TempVar to bool

Convert two fields of struct TempVar in popt.go from uint8 to bool.

This change passes go build -toolexec 'toolstash -cmp' -a std.

Change-Id: I1aa14313e0241a4e9cadd63c6c681ed4e965a9a3
Reviewed-on: https://go-review.googlesource.com/14377Reviewed-by: default avatarBrad Fitzpatrick <bradfitz@golang.org>
Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
parent a20fbe8e
...@@ -523,8 +523,8 @@ type TempVar struct { ...@@ -523,8 +523,8 @@ type TempVar struct {
merge *TempVar // merge var with this one merge *TempVar // merge var with this one
start int64 // smallest Prog.pc in live range start int64 // smallest Prog.pc in live range
end int64 // largest Prog.pc in live range end int64 // largest Prog.pc in live range
addr uint8 // address taken - no accurate end addr bool // address taken - no accurate end
removed uint8 // removed from program removed bool // removed from program
} }
type startcmp []*TempVar type startcmp []*TempVar
...@@ -625,7 +625,7 @@ func mergetemp(firstp *obj.Prog) { ...@@ -625,7 +625,7 @@ func mergetemp(firstp *obj.Prog) {
f.Data = v.use f.Data = v.use
v.use = f v.use = f
if n == p.From.Node && (p.Info.Flags&LeftAddr != 0) { if n == p.From.Node && (p.Info.Flags&LeftAddr != 0) {
v.addr = 1 v.addr = true
} }
} }
} }
...@@ -639,7 +639,7 @@ func mergetemp(firstp *obj.Prog) { ...@@ -639,7 +639,7 @@ func mergetemp(firstp *obj.Prog) {
// Special case. // Special case.
for i := 0; i < len(var_); i++ { for i := 0; i < len(var_); i++ {
v = &var_[i] v = &var_[i]
if v.addr != 0 { if v.addr {
continue continue
} }
...@@ -650,7 +650,7 @@ func mergetemp(firstp *obj.Prog) { ...@@ -650,7 +650,7 @@ func mergetemp(firstp *obj.Prog) {
if p.To.Node == v.node && (p.Info.Flags&RightWrite != 0) && p.Info.Flags&RightRead == 0 { if p.To.Node == v.node && (p.Info.Flags&RightWrite != 0) && p.Info.Flags&RightRead == 0 {
p.As = obj.ANOP p.As = obj.ANOP
p.To = obj.Addr{} p.To = obj.Addr{}
v.removed = 1 v.removed = true
if debugmerge > 0 && Debug['v'] != 0 { if debugmerge > 0 && Debug['v'] != 0 {
fmt.Printf("drop write-only %v\n", v.node.Sym) fmt.Printf("drop write-only %v\n", v.node.Sym)
} }
...@@ -673,7 +673,7 @@ func mergetemp(firstp *obj.Prog) { ...@@ -673,7 +673,7 @@ func mergetemp(firstp *obj.Prog) {
if p.From.Node == v.node && p1.To.Node == v.node && (p.Info.Flags&Move != 0) && (p.Info.Flags|p1.Info.Flags)&(LeftAddr|RightAddr) == 0 && p.Info.Flags&SizeAny == p1.Info.Flags&SizeAny { if p.From.Node == v.node && p1.To.Node == v.node && (p.Info.Flags&Move != 0) && (p.Info.Flags|p1.Info.Flags)&(LeftAddr|RightAddr) == 0 && p.Info.Flags&SizeAny == p1.Info.Flags&SizeAny {
p1.From = p.From p1.From = p.From
Thearch.Excise(f) Thearch.Excise(f)
v.removed = 1 v.removed = true
if debugmerge > 0 && Debug['v'] != 0 { if debugmerge > 0 && Debug['v'] != 0 {
fmt.Printf("drop immediate-use %v\n", v.node.Sym) fmt.Printf("drop immediate-use %v\n", v.node.Sym)
} }
...@@ -695,7 +695,7 @@ func mergetemp(firstp *obj.Prog) { ...@@ -695,7 +695,7 @@ func mergetemp(firstp *obj.Prog) {
for f := v.use; f != nil; f = f.Data.(*Flow) { for f := v.use; f != nil; f = f.Data.(*Flow) {
mergewalk(v, f, uint32(gen)) mergewalk(v, f, uint32(gen))
} }
if v.addr != 0 { if v.addr {
gen++ gen++
for f := v.use; f != nil; f = f.Data.(*Flow) { for f := v.use; f != nil; f = f.Data.(*Flow) {
varkillwalk(v, f, uint32(gen)) varkillwalk(v, f, uint32(gen))
...@@ -727,10 +727,10 @@ func mergetemp(firstp *obj.Prog) { ...@@ -727,10 +727,10 @@ func mergetemp(firstp *obj.Prog) {
for i := 0; i < len(var_); i++ { for i := 0; i < len(var_); i++ {
v = bystart[i] v = bystart[i]
if debugmerge > 0 && Debug['v'] != 0 { if debugmerge > 0 && Debug['v'] != 0 {
fmt.Printf("consider %v: removed=%d\n", Nconv(v.node, obj.FmtSharp), v.removed) fmt.Printf("consider %v: removed=%t\n", Nconv(v.node, obj.FmtSharp), v.removed)
} }
if v.removed != 0 { if v.removed {
continue continue
} }
...@@ -743,7 +743,7 @@ func mergetemp(firstp *obj.Prog) { ...@@ -743,7 +743,7 @@ func mergetemp(firstp *obj.Prog) {
} }
if debugmerge > 0 && Debug['v'] != 0 { if debugmerge > 0 && Debug['v'] != 0 {
fmt.Printf("consider %v: removed=%d nfree=%d nvar=%d\n", Nconv(v.node, obj.FmtSharp), v.removed, nfree, len(var_)) fmt.Printf("consider %v: removed=%t nfree=%d nvar=%d\n", Nconv(v.node, obj.FmtSharp), v.removed, nfree, len(var_))
} }
// Find old temp to reuse if possible. // Find old temp to reuse if possible.
...@@ -791,11 +791,11 @@ func mergetemp(firstp *obj.Prog) { ...@@ -791,11 +791,11 @@ func mergetemp(firstp *obj.Prog) {
for i := 0; i < len(var_); i++ { for i := 0; i < len(var_); i++ {
v = &var_[i] v = &var_[i]
fmt.Printf("var %v %v %d-%d", Nconv(v.node, obj.FmtSharp), v.node.Type, v.start, v.end) fmt.Printf("var %v %v %d-%d", Nconv(v.node, obj.FmtSharp), v.node.Type, v.start, v.end)
if v.addr != 0 { if v.addr {
fmt.Printf(" addr=1") fmt.Printf(" addr=true")
} }
if v.removed != 0 { if v.removed {
fmt.Printf(" dead=1") fmt.Printf(" removed=true")
} }
if v.merge != nil { if v.merge != nil {
fmt.Printf(" merge %v", Nconv(v.merge.node, obj.FmtSharp)) fmt.Printf(" merge %v", Nconv(v.merge.node, obj.FmtSharp))
...@@ -841,7 +841,7 @@ func mergetemp(firstp *obj.Prog) { ...@@ -841,7 +841,7 @@ func mergetemp(firstp *obj.Prog) {
Curfn.Func.Dcl.End = l Curfn.Func.Dcl.End = l
n = l.N n = l.N
v, _ = n.Opt().(*TempVar) v, _ = n.Opt().(*TempVar)
if v != nil && (v.merge != nil || v.removed != 0) { if v != nil && (v.merge != nil || v.removed) {
*lp = l.Next *lp = l.Next
continue continue
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment