[llvm] r225592 - [ADT] Remove the unused default constructor for iterator_range.
Chandler Carruth
chandlerc at gmail.com
Sat Jan 10 17:16:26 PST 2015
Author: chandlerc
Date: Sat Jan 10 19:16:26 2015
New Revision: 225592
URL: http://llvm.org/viewvc/llvm-project?rev=225592&view=rev
Log:
[ADT] Remove the unused default constructor for iterator_range.
This default constructor is a bit weird. It left the range in an invalid
state. That might be reasonable so that you can construct a local
iterator range and assign to it based on some logic to compute the range
you want. If folks would like to support that use case, I can add it
back, but in 238-odd usages none have actually wanted to do this. ;]
Modified:
llvm/trunk/include/llvm/ADT/iterator_range.h
Modified: llvm/trunk/include/llvm/ADT/iterator_range.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/ADT/iterator_range.h?rev=225592&r1=225591&r2=225592&view=diff
==============================================================================
--- llvm/trunk/include/llvm/ADT/iterator_range.h (original)
+++ llvm/trunk/include/llvm/ADT/iterator_range.h Sat Jan 10 19:16:26 2015
@@ -32,7 +32,6 @@ class iterator_range {
IteratorT begin_iterator, end_iterator;
public:
- iterator_range() {}
iterator_range(IteratorT begin_iterator, IteratorT end_iterator)
: begin_iterator(std::move(begin_iterator)),
end_iterator(std::move(end_iterator)) {}
More information about the llvm-commits
mailing list