diff options
author | Duncan P. N. Exon Smith <dexonsmith@apple.com> | 2016-08-18 17:15:25 +0000 |
---|---|---|
committer | Duncan P. N. Exon Smith <dexonsmith@apple.com> | 2016-08-18 17:15:25 +0000 |
commit | 9d748f949977dc972694fb5a9f3a11575521f182 (patch) | |
tree | a14fb0cb48a3ef038f2bbc34c1636938e77c7811 /clang/lib/Serialization/ModuleManager.cpp | |
parent | a8105c607b434a1ae2af35fd4ef0fe46f43c33cf (diff) | |
download | llvm-9d748f949977dc972694fb5a9f3a11575521f182.zip llvm-9d748f949977dc972694fb5a9f3a11575521f182.tar.gz llvm-9d748f949977dc972694fb5a9f3a11575521f182.tar.bz2 |
Reapply "ADT: Remove references in has_rbegin for reverse()"
This reverts commit r279086, reapplying r279084. I'm not sure what I
ran before, because the compile failure for ADTTests reproduced locally.
The problem is that TestRev is calling BidirectionalVector::rbegin()
when the BidirectionalVector is const, but rbegin() is always non-const.
I've updated BidirectionalVector::rbegin() to be callable from const.
Original commit message follows.
--
As a follow-up to r278991, add some tests that check that
decltype(reverse(R).begin()) == decltype(R.rbegin()), and get them
passing by adding std::remove_reference to has_rbegin.
I'm using static_assert instead of EXPECT_TRUE (and updated the other
has_rbegin check from r278991 in the same way) since I figure that's
more helpful.
llvm-svn: 279091
Diffstat (limited to 'clang/lib/Serialization/ModuleManager.cpp')
0 files changed, 0 insertions, 0 deletions