[llvm] r354469 - Fix the build with gcc/libstdc++ 4.8.2 after r354441

Hans Wennborg via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 20 06:57:53 PST 2019


I guess building with 4.8.2 isn't really supported anymore (or is it
just deprecated?), but my project hasn't quite moved off it yet.

On Wed, Feb 20, 2019 at 3:51 PM Roman Lebedev via llvm-commits
<llvm-commits at lists.llvm.org> wrote:
>
> Hm, thanks, no bots complained about that.
>
> On Wed, Feb 20, 2019 at 5:49 PM Hans Wennborg via llvm-commits
> <llvm-commits at lists.llvm.org> wrote:
> >
> > Author: hans
> > Date: Wed Feb 20 06:50:08 2019
> > New Revision: 354469
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=354469&view=rev
> > Log:
> > Fix the build with gcc/libstdc++ 4.8.2 after r354441
> >
> > Modified:
> >     llvm/trunk/tools/llvm-exegesis/lib/Clustering.cpp
> >
> > Modified: llvm/trunk/tools/llvm-exegesis/lib/Clustering.cpp
> > URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-exegesis/lib/Clustering.cpp?rev=354469&r1=354468&r2=354469&view=diff
> > ==============================================================================
> > --- llvm/trunk/tools/llvm-exegesis/lib/Clustering.cpp (original)
> > +++ llvm/trunk/tools/llvm-exegesis/lib/Clustering.cpp Wed Feb 20 06:50:08 2019
> > @@ -226,10 +226,10 @@ void InstructionBenchmarkClustering::sta
> >                        ClusterIdForPoint_[P] = UnstableCluster.Id;
> >                      });
> >        // Actually append to-be-moved points to the new cluster.
> > -      UnstableCluster.PointIndices.insert(UnstableCluster.PointIndices.cend(),
> > +      UnstableCluster.PointIndices.insert(UnstableCluster.PointIndices.end(),
> >                                            it, OldCluster.PointIndices.end());
> >        // And finally, remove "to-be-moved" points form the old cluster.
> > -      OldCluster.PointIndices.erase(it, OldCluster.PointIndices.cend());
> > +      OldCluster.PointIndices.erase(it, OldCluster.PointIndices.end());
> >        // Now, the old cluster may end up being empty, but let's just keep it
> >        // in whatever state it ended up. Purging empty clusters isn't worth it.
> >      };
> >
> >
> > _______________________________________________
> > llvm-commits mailing list
> > llvm-commits at lists.llvm.org
> > https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits


More information about the llvm-commits mailing list