Subzero: Minor refactoring/additions in preparation for phi edge splitting.

Subzero: Minor refactoring/additions in preparation for phi edge splitting.

The only functional change (though not actually visible at this point) is that redundant assignment elimination is moved into a separate pass.

BUG= none
R=jvoung@chromium.org

Committed: https://gerrit.chromium.org/gerrit/gitweb?p=native_client/pnacl-subzero.git;a=commit;h=98712a39588f258eba8e6448a53a182de4fd1c3c

[Jim Stichnoth, 2014-10-24 17:28:25]

stichnot@chromium.org changed reviewers:
+ jvoung@chromium.org

[jvoung (off chromium), 2014-10-24 17:50:40]

otherwise LGTM

https://codereview.chromium.org/672393003/diff/20001/src/IceTargetLowering.h
File src/IceTargetLowering.h (right):

https://codereview.chromium.org/672393003/diff/20001/src/IceTargetLowering.h#...
src/IceTargetLowering.h:93: void advanceBackward(InstList::iterator &I) const;
Can this be removed as part of the CL?

[Jim Stichnoth, 2014-10-24 17:58:53]

https://codereview.chromium.org/672393003/diff/20001/src/IceTargetLowering.h
File src/IceTargetLowering.h (right):

https://codereview.chromium.org/672393003/diff/20001/src/IceTargetLowering.h#...
src/IceTargetLowering.h:93: void advanceBackward(InstList::iterator &I) const;
On 2014/10/24 17:50:40, jvoung wrote:
> Can this be removed as part of the CL?

Done.

[Jim Stichnoth, 2014-10-24 17:59:06]

Committed patchset #3 (id:40001) manually as
98712a39588f258eba8e6448a53a182de4fd1c3c (presubmit successful).