diff options
| author | Russ Cox <rsc@golang.org> | 2014-07-16 19:26:36 -0400 |
|---|---|---|
| committer | Russ Cox <rsc@golang.org> | 2014-07-16 19:26:36 -0400 |
| commit | ac3b89c41e231815a8355fbeac5dec43d8b27504 (patch) | |
| tree | 7098735c4187148e46a6a56f810017b155da2d9f /src/liblink/pcln.c | |
| parent | 5180f53088de94c879eccd80311877f1541e3ed1 (diff) | |
| download | go-ac3b89c41e231815a8355fbeac5dec43d8b27504.tar.gz | |
liblink: fix precedence bug in pcdata comparison
As written, the ! applies before the &1.
This would crash writing out missing pcdata tables
if we ever used non-contiguous IDs in a function.
We don't, but fix anyway.
LGTM=iant, minux
R=minux, iant
CC=golang-codereviews
https://codereview.appspot.com/117810047
Diffstat (limited to 'src/liblink/pcln.c')
| -rw-r--r-- | src/liblink/pcln.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/liblink/pcln.c b/src/liblink/pcln.c index 4b2b85543..f0ee1dc67 100644 --- a/src/liblink/pcln.c +++ b/src/liblink/pcln.c @@ -279,7 +279,7 @@ linkpcln(Link *ctxt, LSym *cursym) } // pcdata. for(i=0; i<npcdata; i++) { - if(!(havepc[i/32]>>(i%32))&1) + if(((havepc[i/32]>>(i%32))&1) == 0) continue; funcpctab(ctxt, &pcln->pcdata[i], cursym, "pctopcdata", pctopcdata, (void*)(uintptr)i); } |