tree d30df71e67996d0cf88687488bd73a41d1f02551
parent 7c9728faead1368150bcaa8313bc460df29ea1b7
author Jim Stichnoth <stichnot@chromium.org> 1472682752 -0700
committer Jim Stichnoth <stichnot@chromium.org> 1472682752 -0700

Subzero: Deterministically sort local var stack offsets.

Currently, TargetLowering::sortVarsByAlignment() uses the variable's required alignment as the primary key, but then std::sort() breaks ties arbitrarily.  This can give different results across different STL implementations, for example when building against LLVM trunk versus building in the PNaCl environment.

The fix is to use Variable::Number as the secondary key.

BUG= none
R=kschimpf@google.com

Review URL: https://codereview.chromium.org/2295393002 .