Page MenuHomePhabricator

Fix nondeterminism introduced in r353954
ClosedPublic

Authored by kongyi on Mar 25 2019, 6:45 PM.

Details

Summary

DenseMap iteration order is not guaranteed, use MapVector instead.

Fix by srhines.

Diff Detail

Repository
rL LLVM

Event Timeline

kongyi created this revision.Mar 25 2019, 6:45 PM
srhines accepted this revision.Mar 25 2019, 11:43 PM

@jmorse can I get your approval on this CL before we submit it? Thanks.

This revision is now accepted and ready to land.Mar 25 2019, 11:43 PM
jmorse accepted this revision.Mar 26 2019, 3:29 AM

LGTM, many thanks for the find and fix!

This revision was automatically updated to reflect the committed changes.