[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