[llvm] r294357 - Fix some missing negations in the traits checking from r294349
Duncan P. N. Exon Smith via llvm-commits
llvm-commits at lists.llvm.org
Tue Feb 7 14:45:44 PST 2017
Thanks David, I just saw the buildbot emails now.
I take the blame, but an interesting behaviour in the monorepo `git llvm push` script made it easy for me to make a mistake. I found the same failures locally, fixed them to get `ninja check` clean, and then pushed... neglecting to amend my commit.
Mehdi and Justin: should `git llvm push` fail if there are changes in the checkout?
> On 2017-Feb-07, at 13:31, David Blaikie via llvm-commits <llvm-commits at lists.llvm.org> wrote:
>
> Author: dblaikie
> Date: Tue Feb 7 15:31:03 2017
> New Revision: 294357
>
> URL: http://llvm.org/viewvc/llvm-project?rev=294357&view=rev
> Log:
> Fix some missing negations in the traits checking from r294349
>
> Modified:
> llvm/trunk/unittests/ADT/IListIteratorTest.cpp
>
> Modified: llvm/trunk/unittests/ADT/IListIteratorTest.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/unittests/ADT/IListIteratorTest.cpp?rev=294357&r1=294356&r2=294357&view=diff
> ==============================================================================
> --- llvm/trunk/unittests/ADT/IListIteratorTest.cpp (original)
> +++ llvm/trunk/unittests/ADT/IListIteratorTest.cpp Tue Feb 7 15:31:03 2017
> @@ -159,15 +159,15 @@ TEST(IListIteratorTest, ReverseConstruct
> EXPECT_EQ(CL.rbegin(), const_reverse_iterator(CL.end()));
>
> // Confirm lack of implicit conversions.
> - static_assert(std::is_convertible<iterator, reverse_iterator>::value,
> + static_assert(!std::is_convertible<iterator, reverse_iterator>::value,
> "unexpected implicit conversion");
> - static_assert(std::is_convertible<reverse_iterator, iterator>::value,
> + static_assert(!std::is_convertible<reverse_iterator, iterator>::value,
> "unexpected implicit conversion");
> static_assert(
> - std::is_convertible<const_iterator, const_reverse_iterator>::value,
> + !std::is_convertible<const_iterator, const_reverse_iterator>::value,
> "unexpected implicit conversion");
> static_assert(
> - std::is_convertible<const_reverse_iterator, const_iterator>::value,
> + !std::is_convertible<const_reverse_iterator, const_iterator>::value,
> "unexpected implicit conversion");
> }
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
More information about the llvm-commits
mailing list