5 commits

Author	SHA1	Message	Date
Ian Lance Taylor	72655afb4e	cmd/compile: remove NodeList type ... That was easy. Fixes #14473. Change-Id: I9d1d20a5c5a9b1423e6c72c0460ee4a78130864f Reviewed-on: https://go-review.googlesource.com/20521 Run-TryBot: Ian Lance Taylor <iant@golang.org> Reviewed-by: David Crawshaw <crawshaw@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org>	2016-03-10 18:22:07 +00:00
Ian Lance Taylor	7e92c86df2	cmd/compile: remove listsort ... The listsort function is no longer used, except in a test. Change the test to use sort.Sort instead. Change-Id: Ib634705cc1bc3b1d8fc3795bd4ed2894e6abc284 Reviewed-on: https://go-review.googlesource.com/19964 Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>	2016-03-03 19:50:23 +00:00
Matthew Dempsky	5609a48931	cmd/compile: make cmpstackvarlt properly asymmetric ... Previously, given two Nodes n1 and n2 of different non-PAUTO classes (e.g., PPARAM and PPARAMOUT), cmpstackvarlt(n1, n2) and cmpstackvarlt(n2, n1) both returned true, which is nonsense. This doesn't seem to cause any visible miscompilation problems, but notably fixing it does cause toolstash/buildall to fail. Change-Id: I33b2c66e902c5eced875d8fbf18b7cfdc81e8aed Reviewed-on: https://go-review.googlesource.com/19778 Run-TryBot: Matthew Dempsky <mdempsky@google.com> Reviewed-by: Ian Lance Taylor <iant@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org>	2016-02-22 00:09:36 +00:00
Håvard Haugen	6d0178359f	cmd/compile/internal/gc: eliminate stringsCompare for stackvar sorting ... Passes go build -a -toolexec 'toolstash -cmp' std cmd. Change-Id: I2a87d31da74affdf3d0f358d0efdb3f1c646d917 Reviewed-on: https://go-review.googlesource.com/14759 Reviewed-by: Dave Cheney <dave@cheney.net>	2015-09-20 21:07:12 +00:00
Håvard Haugen	f482a0f023	cmd/compile/internal/gc: add unit test for cmpstackvar ... A followup CL will rewrite listsort to use the new cmpstackvarlt and change cmpstackvar to avoid stringsCompare. Change-Id: Idf0857a3bd67f9e2243ba82aa0bff510612927c3 Reviewed-on: https://go-review.googlesource.com/14611 Reviewed-by: Dave Cheney <dave@cheney.net>	2015-09-17 23:46:38 +00:00