From ffc5c1f3b88aa715b7804387746f83d71a132b5e Mon Sep 17 00:00:00 2001 From: Chandler Carruth Date: Sun, 11 Jan 2015 01:16:26 +0000 Subject: [PATCH] [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. ;] llvm-svn: 225592 --- llvm/include/llvm/ADT/iterator_range.h | 1 - 1 file changed, 1 deletion(-) diff --git a/llvm/include/llvm/ADT/iterator_range.h b/llvm/include/llvm/ADT/iterator_range.h index ecaf4a2..523a86f 100644 --- a/llvm/include/llvm/ADT/iterator_range.h +++ b/llvm/include/llvm/ADT/iterator_range.h @@ -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)) {} -- 2.7.4