Commit 25fc842f authored by Josh Bleecher Snyder's avatar Josh Bleecher Snyder

cmd/compile: remove forceObjFileStability

The textual import/export format is ancient history.

Change-Id: Iebe90bfd9bd3074eb191186d86e5f4286ce3b1f3
Reviewed-on: https://go-review.googlesource.com/39850
Run-TryBot: Josh Bleecher Snyder <josharian@gmail.com>
Reviewed-by: default avatarMatthew Dempsky <mdempsky@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
parent 91433eb5
...@@ -133,12 +133,6 @@ import ( ...@@ -133,12 +133,6 @@ import (
// (suspected) format errors, and whenever a change is made to the format. // (suspected) format errors, and whenever a change is made to the format.
const debugFormat = false // default: false const debugFormat = false // default: false
// forceObjFileStability enforces additional constraints in export data
// and other parts of the compiler to eliminate object file differences
// only due to the choice of export format.
// TODO(gri) disable and remove once there is only one export format again
const forceObjFileStability = true
// Current export format version. Increase with each format change. // Current export format version. Increase with each format change.
// 4: type name objects support type aliases, uses aliasTag // 4: type name objects support type aliases, uses aliasTag
// 3: Go1.8 encoding (same as version 2, aliasTag defined but never used) // 3: Go1.8 encoding (same as version 2, aliasTag defined but never used)
...@@ -979,7 +973,7 @@ func parName(f *Field, numbered bool) string { ...@@ -979,7 +973,7 @@ func parName(f *Field, numbered bool) string {
// Functions that can be inlined use numbered parameters so we can distinguish them // Functions that can be inlined use numbered parameters so we can distinguish them
// from other names in their context after inlining (i.e., the parameter numbering // from other names in their context after inlining (i.e., the parameter numbering
// is a form of parameter rewriting). See issue 4326 for an example and test case. // is a form of parameter rewriting). See issue 4326 for an example and test case.
if forceObjFileStability || numbered { if numbered {
if !strings.Contains(name, "·") && f.Nname != nil && f.Nname.Name != nil && f.Nname.Name.Vargen > 0 { if !strings.Contains(name, "·") && f.Nname != nil && f.Nname.Name != nil && f.Nname.Name.Vargen > 0 {
name = fmt.Sprintf("%s·%d", name, f.Nname.Name.Vargen) // append Vargen name = fmt.Sprintf("%s·%d", name, f.Nname.Name.Vargen) // append Vargen
} }
......
...@@ -1501,11 +1501,6 @@ func dumptypestructs() { ...@@ -1501,11 +1501,6 @@ func dumptypestructs() {
} }
// generate import strings for imported packages // generate import strings for imported packages
if forceObjFileStability {
// Sorting the packages is not necessary but to compare binaries created
// using textual and binary format we sort by path to reduce differences.
sort.Sort(pkgByPath(pkgs))
}
for _, p := range pkgs { for _, p := range pkgs {
if p.Direct { if p.Direct {
dimportpath(p) dimportpath(p)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment