diff options
| author | Kunpei Sakai <namusyaka@gmail.com> | 2018-02-25 20:08:22 +0900 |
|---|---|---|
| committer | Matthew Dempsky <mdempsky@google.com> | 2018-02-26 20:22:06 +0000 |
| commit | 0c471dfae29b0759c6f12e32d2d5ca69ea79d5d8 (patch) | |
| tree | a1f155bc179ab634ddbfc5639b37275aa7b36f58 /src/cmd/internal | |
| parent | f4d9c309018f6bbb85c7076a9fbe0387fb7b0b1c (diff) | |
| download | go-0c471dfae29b0759c6f12e32d2d5ca69ea79d5d8.tar.xz | |
cmd: avoid unnecessary type conversions
CL generated mechanically with github.com/mdempsky/unconvert.
Also updated cmd/compile/internal/ssa/gen/*.rules manually.
Change-Id: If721ef73cf0771ae83ce7e2d11623fc8d9155768
Reviewed-on: https://go-review.googlesource.com/97075
Reviewed-by: Matthew Dempsky <mdempsky@google.com>
Run-TryBot: Matthew Dempsky <mdempsky@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Diffstat (limited to 'src/cmd/internal')
| -rw-r--r-- | src/cmd/internal/buildid/note.go | 2 | ||||
| -rw-r--r-- | src/cmd/internal/dwarf/dwarf.go | 4 | ||||
| -rw-r--r-- | src/cmd/internal/goobj/read.go | 2 | ||||
| -rw-r--r-- | src/cmd/internal/obj/arm/obj5.go | 2 | ||||
| -rw-r--r-- | src/cmd/internal/obj/arm64/asm7.go | 14 | ||||
| -rw-r--r-- | src/cmd/internal/obj/ppc64/obj9.go | 2 | ||||
| -rw-r--r-- | src/cmd/internal/obj/x86/asm6.go | 2 | ||||
| -rw-r--r-- | src/cmd/internal/objfile/goobj.go | 2 |
8 files changed, 15 insertions, 15 deletions
diff --git a/src/cmd/internal/buildid/note.go b/src/cmd/internal/buildid/note.go index f0439fb0bf..5895da906a 100644 --- a/src/cmd/internal/buildid/note.go +++ b/src/cmd/internal/buildid/note.go @@ -147,7 +147,7 @@ func readELF(name string, f *os.File, data []byte) (buildid string, err error) { break } off += notesz - align := uint64(p.Align) + align := p.Align alignedOff := (off + align - 1) &^ (align - 1) notesz += alignedOff - off off = alignedOff diff --git a/src/cmd/internal/dwarf/dwarf.go b/src/cmd/internal/dwarf/dwarf.go index 478a75c4a7..b9cf95bf75 100644 --- a/src/cmd/internal/dwarf/dwarf.go +++ b/src/cmd/internal/dwarf/dwarf.go @@ -776,7 +776,7 @@ func GetAbbrev() []byte { // See section 7.5.3 buf = AppendUleb128(buf, uint64(i)) buf = AppendUleb128(buf, uint64(abbrevs[i].tag)) - buf = append(buf, byte(abbrevs[i].children)) + buf = append(buf, abbrevs[i].children) for _, f := range abbrevs[i].attr { buf = AppendUleb128(buf, uint64(f.attr)) buf = AppendUleb128(buf, uint64(f.form)) @@ -1454,7 +1454,7 @@ func putvar(ctxt Context, s *FnState, v *Var, absfn Sym, fnabbrev, inlIndex int, } if abbrevUsesLoclist(abbrev) { - putattr(ctxt, s.Info, abbrev, DW_FORM_sec_offset, DW_CLS_PTR, int64(s.Loc.Len()), s.Loc) + putattr(ctxt, s.Info, abbrev, DW_FORM_sec_offset, DW_CLS_PTR, s.Loc.Len(), s.Loc) v.PutLocationList(s.Loc, s.StartPC) } else { loc := encbuf[:0] diff --git a/src/cmd/internal/goobj/read.go b/src/cmd/internal/goobj/read.go index ebdc37575f..e39180cad6 100644 --- a/src/cmd/internal/goobj/read.go +++ b/src/cmd/internal/goobj/read.go @@ -585,7 +585,7 @@ func (r *objReader) parseObject(prefix []byte) error { f.FuncData[i].Sym = r.readSymID() } for i := range f.FuncData { - f.FuncData[i].Offset = int64(r.readInt()) // TODO + f.FuncData[i].Offset = r.readInt() // TODO } f.File = make([]string, r.readInt()) for i := range f.File { diff --git a/src/cmd/internal/obj/arm/obj5.go b/src/cmd/internal/obj/arm/obj5.go index f7d0f17168..2046649e38 100644 --- a/src/cmd/internal/obj/arm/obj5.go +++ b/src/cmd/internal/obj/arm/obj5.go @@ -346,7 +346,7 @@ func preprocess(ctxt *obj.Link, cursym *obj.LSym, newprog obj.ProgAlloc) { o := p.As switch o { case obj.ATEXT: - autosize = int32(autoffset) + autosize = autoffset if p.Mark&LEAF != 0 && autosize == 0 { // A leaf function with no locals has no frame. diff --git a/src/cmd/internal/obj/arm64/asm7.go b/src/cmd/internal/obj/arm64/asm7.go index 9d064806a1..a8b2e1d39e 100644 --- a/src/cmd/internal/obj/arm64/asm7.go +++ b/src/cmd/internal/obj/arm64/asm7.go @@ -861,7 +861,7 @@ func (c *ctxt7) addpool(p *obj.Prog, a *obj.Addr) { // MOVD addr, REGTMP // MOVD REGTMP, R // where addr is the address of the DWORD containing the address of foo. - if p.As == AMOVD && a.Type != obj.TYPE_MEM || cls == C_ADDR || cls == C_VCON || int64(lit) != int64(int32(lit)) || uint64(lit) != uint64(uint32(lit)) { + if p.As == AMOVD && a.Type != obj.TYPE_MEM || cls == C_ADDR || cls == C_VCON || lit != int64(int32(lit)) || uint64(lit) != uint64(uint32(lit)) { // conservative: don't know if we want signed or unsigned extension. // in case of ambiguity, store 64-bit t.As = ADWORD @@ -2838,7 +2838,7 @@ func (c *ctxt7) asmout(p *obj.Prog, o *Optab, out []uint32) { if v == 0 { c.ctxt.Diag("illegal system register:\n%v", p) } - if (o1 & uint32(v&^(3<<19))) != 0 { + if (o1 & (v &^ (3 << 19))) != 0 { c.ctxt.Diag("MRS register value overlap\n%v", p) } @@ -2858,7 +2858,7 @@ func (c *ctxt7) asmout(p *obj.Prog, o *Optab, out []uint32) { if v == 0 { c.ctxt.Diag("illegal system register:\n%v", p) } - if (o1 & uint32(v&^(3<<19))) != 0 { + if (o1 & (v &^ (3 << 19))) != 0 { c.ctxt.Diag("MSR register value overlap\n%v", p) } @@ -3360,7 +3360,7 @@ func (c *ctxt7) asmout(p *obj.Prog, o *Optab, out []uint32) { size = 2 } else if p.As == AVAND || p.As == AVEOR { size = 0 - } else if (p.As == AVFMLA || p.As == AVFMLS) { + } else if p.As == AVFMLA || p.As == AVFMLS { if af == ARNG_2D { size = 1 } else { @@ -3512,7 +3512,7 @@ func (c *ctxt7) asmout(p *obj.Prog, o *Optab, out []uint32) { rt := int(p.To.Reg) imm5 := 0 o1 = 1<<30 | 7<<25 | 7<<10 - index :=int(p.From.Index) + index := int(p.From.Index) switch (p.To.Reg >> 5) & 15 { case ARNG_B: c.checkindex(p, index, 15) @@ -3662,7 +3662,7 @@ func (c *ctxt7) asmout(p *obj.Prog, o *Optab, out []uint32) { default: c.ctxt.Diag("invalid arrangement on VMOV Rn, Vd.<T>: %v\n", p) } - o1 |= (uint32(Q&1) << 30) | (uint32(imm5&0x1f) << 16) + o1 |= (Q & 1 << 30) | (imm5 & 0x1f << 16) o1 |= (uint32(rf&31) << 5) | uint32(rt&31) case 83: /* vmov Vn.<T>, Vd.<T> */ @@ -3883,7 +3883,7 @@ func (c *ctxt7) asmout(p *obj.Prog, o *Optab, out []uint32) { } o1 = c.opldrpp(p, p.As) - o1 |= (uint32(r&31) << 5) | (uint32((imm>>3)&0xfff) << 10) | (uint32(v & 31)) + o1 |= (uint32(r&31) << 5) | ((imm >> 3) & 0xfff << 10) | (v & 31) } out[0] = o1 diff --git a/src/cmd/internal/obj/ppc64/obj9.go b/src/cmd/internal/obj/ppc64/obj9.go index c50cd3b06c..c468ee93a4 100644 --- a/src/cmd/internal/obj/ppc64/obj9.go +++ b/src/cmd/internal/obj/ppc64/obj9.go @@ -524,7 +524,7 @@ func preprocess(ctxt *obj.Link, cursym *obj.LSym, newprog obj.ProgAlloc) { q.To.Type = obj.TYPE_MEM q.To.Offset = int64(-autosize) q.To.Reg = REGSP - q.Spadj = int32(autosize) + q.Spadj = autosize } else { // Frame size is too large for a MOVDU instruction. // Store link register before decrementing SP, so if a signal comes diff --git a/src/cmd/internal/obj/x86/asm6.go b/src/cmd/internal/obj/x86/asm6.go index b14d1d232b..7c666e8f6e 100644 --- a/src/cmd/internal/obj/x86/asm6.go +++ b/src/cmd/internal/obj/x86/asm6.go @@ -3930,7 +3930,7 @@ func (asmbuf *AsmBuf) doasm(ctxt *obj.Link, cursym *obj.LSym, p *obj.Prog) { } else { regnum &= byte(p.GetFrom3().Reg - REG_Y0) } - asmbuf.Put1(byte(o.op[z+2]) | regnum) + asmbuf.Put1(o.op[z+2] | regnum) asmbuf.Put1(byte(p.From.Offset)) case Zvex_i_rm_v_r: diff --git a/src/cmd/internal/objfile/goobj.go b/src/cmd/internal/objfile/goobj.go index 51fa6e873f..473e773ec2 100644 --- a/src/cmd/internal/objfile/goobj.go +++ b/src/cmd/internal/objfile/goobj.go @@ -64,7 +64,7 @@ func (f *goobjFile) symbols() ([]Sym, error) { var syms []Sym for _, s := range f.goobj.Syms { seen[s.SymID] = true - sym := Sym{Addr: uint64(s.Data.Offset), Name: goobjName(s.SymID), Size: int64(s.Size), Type: s.Type.Name, Code: '?'} + sym := Sym{Addr: uint64(s.Data.Offset), Name: goobjName(s.SymID), Size: s.Size, Type: s.Type.Name, Code: '?'} switch s.Kind { case objabi.STEXT: sym.Code = 'T' |
