Navigation Menu

Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

cmd/compile: output variable names in data race reports #4246

Open
dvyukov opened this issue Oct 15, 2012 · 4 comments
Open

cmd/compile: output variable names in data race reports #4246

dvyukov opened this issue Oct 15, 2012 · 4 comments
Labels
RaceDetector
Milestone
Unplanned

Comments

@dvyukov
Copy link
Member

dvyukov commented Oct 15, 2012 

It would be nice if in data race reports we print variable size and symbolic name as
well. E.g. "Read of 'foo.Bar'".
@rsc
Copy link
Contributor

rsc commented Dec 4, 2013

Comment 1:

Labels changed: added repo-main.

@rsc
Copy link
Contributor

rsc commented Mar 3, 2014

Comment 2:

Adding Release=None to all Priority=Someday bugs.

Labels changed: added release-none.

@rsc rsc added this to the Unplanned milestone Apr 10, 2015
@rsc rsc changed the title cmd/gc: output variable names in data race reports cmd/compile: output variable names in data race reports Jun 8, 2015
@odeke-em
Copy link
Member

odeke-em commented Oct 3, 2016

@dvyukov could you help me with a play snippet repro for this bug?

The reason am asking is because I thought we were dealing with data races at runtime but since this title is for a compiler bug I'd like to take a stab at it.
Thanks.

@dvyukov
Copy link
Member Author

dvyukov commented Oct 4, 2016

@odeke-em
Just any racy program will do.
Say in this program:
https://play.golang.org/p/suCIOboNMX
we could say that the write racy expression is 'x = 42' and read racy expression is 'x'.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
RaceDetector
Projects
None yet
Milestone
Unplanned
Development

No branches or pull requests
3 participants
@rsc @dvyukov @odeke-em