[PATCH] D66259: Migrate llvm::make_unique to std::make_unique
JF Bastien via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 15 10:18:53 PDT 2019
jfb added a comment.
In D66259#1631802 <https://reviews.llvm.org/D66259#1631802>, @jwakely wrote:
> In D66259#1631586 <https://reviews.llvm.org/D66259#1631586>, @thakis wrote:
>
> > It looks like this breaks building with gcc5.1:
>
>
> Why are you using 5.1? That's dumb. There are tons of bugs in 5.1 that are fixed in later 5.x releases, especially if you're using C++14.
>
> That's a really bad choice of minimum version to insist on supporting.
>
> Source: me, because I broke lots of stuff in 5.1 and then fixed it.
I'll bring this up on the list and suggest we move to 5.5 as you suggested privately.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D66259/new/
https://reviews.llvm.org/D66259
More information about the llvm-commits
mailing list