tree 2f480a254d2e159199e07184fc0b0e5e203c7c4f
parent d2daa94edeca88281796524fae45f38f11a7bef2
author John Stiles <johnstiles@google.com> 1600803616 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> 1600806019 +0000

Fix nondeterminism when copying intrinsic functions.

Previously, we copied intrinsic functions in a totally arbitrary order;
it used an unordered_set of pointers, so it could be affected by
switching standard libraries OR by malloc nondeterminism. (Surprisingly,
it was fairly consistent in practice on OS X/Linux.) This CL sorts the
intrinsic functions into a consistent order before copying them.

Change-Id: If90342bb77a9ae237a3ce91be3a9847311a722c4
Bug: skia:10749
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/318700
Commit-Queue: John Stiles <johnstiles@google.com>
Auto-Submit: John Stiles <johnstiles@google.com>
Reviewed-by: Ethan Nicholas <ethannicholas@google.com>