[ADT] Remove the unused default constructor for iterator_range.
authorChandler Carruth <chandlerc@gmail.com>
Sun, 11 Jan 2015 01:16:26 +0000 (01:16 +0000)
committerChandler Carruth <chandlerc@gmail.com>
Sun, 11 Jan 2015 01:16:26 +0000 (01:16 +0000)
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. ;]

llvm-svn: 225592

llvm/include/llvm/ADT/iterator_range.h

index ecaf4a2..523a86f 100644 (file)
@@ -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)) {}