You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
cmd/compile currently allows using the -race and -msan flags on any architecture, but there's only runtime support on amd64. Further, -race support wouldn't current work on LR architectures even with runtime support, because racewalk.go hardcodes assumptions about where to find the caller's PC.
Another complication is there are a handful of regress tests that explicitly set -race. A recent refactoring (CL 99416) caused these tests to break, because I assumed no one would be using -race on an unsupported platform.
For comparison, the -dynlink and -shared flags are only usable on platforms where they're supported.
My personal preference would be to make setting -race or -msan an error on !amd64 arches.
@cherrymui suggested CL 99676 to make -race a silent no-op on !amd64.
@ianlancetaylor Thanks for pointing that out. Yes, I meant making it an error. (Edited comment to fix that.)
spf13
added
the
NeedsFix
The path to resolution is known, but the work has not been done.
label
Mar 26, 2018
gopherbot
removed
the
NeedsDecision
Feedback is required from experts, contributors, and/or the community before a change can be made.
label
Mar 26, 2018
spf13
changed the title
cmd/compile: -race and -msan on non-amd64 arches
cmd/compile: reject -race and -msan on non-amd64 arches
Mar 26, 2018
cmd/compile currently allows using the -race and -msan flags on any architecture, but there's only runtime support on amd64. Further, -race support wouldn't current work on LR architectures even with runtime support, because racewalk.go hardcodes assumptions about where to find the caller's PC.
Another complication is there are a handful of regress tests that explicitly set
-race
. A recent refactoring (CL 99416) caused these tests to break, because I assumed no one would be using-race
on an unsupported platform.For comparison, the -dynlink and -shared flags are only usable on platforms where they're supported.
My personal preference would be to make setting -race or -msan an error on !amd64 arches.
@cherrymui suggested CL 99676 to make -race a silent no-op on !amd64.
/cc @aclements @ianlancetaylor
The text was updated successfully, but these errors were encountered: