FAQ
Reviewers: remy_archlinux.org,

Message:
Hello remy@archlinux.org (cc: golang-dev@googlegroups.com,
rsc@golang.org),

I'd like you to review this change to
https://code.google.com/p/go


Description:
6g: delete unnecessary OXXX initialization.

No longer necessary after https://codereview.appspot.com/6497073/
removed the `if(n5.op != OXXX) { regfree(&n5); }`.

Please review this at http://codereview.appspot.com/6498101/

Affected files:
M src/cmd/6g/cgen.c


Index: src/cmd/6g/cgen.c
===================================================================
--- a/src/cmd/6g/cgen.c
+++ b/src/cmd/6g/cgen.c
@@ -669,7 +669,6 @@

if(!debug['B'] && !n->bounded) {
// check bounds
- n5.op = OXXX;
t = types[TUINT32];
if(is64(nr->type))
t = types[TUINT64];

Search Discussions

Related Discussions

Discussion Navigation
viewthread | post
Discussion Overview
groupgolang-dev @
categoriesgo
postedSep 7, '12 at 6:03a
activeSep 10, '12 at 1:24a
posts3
users2
websitegolang.org

2 users in discussion

Nigeltao: 2 posts R: 1 post

People

Translate

site design / logo © 2022 Grokbase