[llvm-commits] [PATCH, PowerPC] Make TOC contents deterministic

Ulrich Weigand Ulrich.Weigand at de.ibm.com
Mon Nov 12 11:15:32 PST 2012


Hal Finkel <hfinkel at anl.gov> wrote on 12.11.2012 20:00:20:
> ----- Original Message -----
> > From: "Ulrich Weigand" <Ulrich.Weigand at de.ibm.com>
> > The attached patch fixes this problem by replacing the DenseMap data
> > structure with a MapVector, which provides bascially the same API
> > except
> > that the iterator is guaranteed to use a deterministic sequence
> > (namely the
> > sequence in which symbols have been added to the TOC).   It would
> > appear
> > this data structure was added specifically to solve this type of
> > problem,
> > and indeed it does work for the case of the TOC.
> >
> > OK to commit?
>
> I love two-line changes ;) Yes, please commit.

Checked in as revision 167737.

Thanks,
Ulrich




More information about the llvm-commits mailing list