]> Cypherpunks repositories - gostls13.git/commit
cmd/compile: adapt GOSSAHASH to set a DebugTest flag in ssa.Config
authorDavid Chase <drchase@google.com>
Thu, 15 Sep 2016 20:51:35 +0000 (16:51 -0400)
committerDavid Chase <drchase@google.com>
Thu, 15 Sep 2016 21:49:49 +0000 (21:49 +0000)
commit897c0ebf61daaaf9c02be577f1afcb2c67ff3ed5
treea19e46a1144ed1a5aae5bb45cbd836440b05831a
parentb265d51789ea7ccf68787ebb30a2cdf03cc9d4fe
cmd/compile: adapt GOSSAHASH to set a DebugTest flag in ssa.Config

Binary search remains our friend.
Suppose you add an ought-to-be-benign pattern to PPC64.rules,
and make.bash starts crashing.  You can guard the pattern(s)
with config.DebugTest:

(Eq8 x y) && config.DebugTest && isSigned(x.Type) &&
   isSigned(y.Type) ->
   (Equal (CMPW (SignExt8to32 x) (SignExt8to32 y)))

and then

  gossahash -s ./make.bash
  ...
  (go drink beer while silicon minions toil)
  ...
  Trying ./make.bash args=[], env=[GOSSAHASH=100110010111110]
  ./make.bash failed (1 distinct triggers): exit status 1
  Trigger string is 'GOSSAHASH triggered (*importReader).readByte',
    repeated 1 times
  Review GSHS_LAST_FAIL.0.log for failing run
  Finished with GOSSAHASH=100110010111110

Change-Id: I4eff46ebaf496baa2acedd32e217005cb3ac1c62
Reviewed-on: https://go-review.googlesource.com/29273
Run-TryBot: David Chase <drchase@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Keith Randall <khr@golang.org>
src/cmd/compile/internal/gc/ssa.go
src/cmd/compile/internal/ssa/config.go