<div dir="ltr">On Thu, Aug 8, 2013 at 6:14 AM, Edwin Vane <span dir="ltr"><<a href="mailto:edwin.vane@intel.com" target="_blank">edwin.vane@intel.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
  1. I was concerned that although `Range` describes what I want that my use of it to describe an offset and number of consecutive items in a vector was against it's intended use as a **source** range.<br></blockquote>
<div><br></div><div>Ooooh. Ok, I assumed the output of vector<Range> conflict would be overlapping *source* ranges (apparently I didn't read closely enough, no idea how I convinced myself that this makes sense :P)</div>
<div><br></div><div>In that case I'd suggest to either:</div><div>- use pair</div><div>- report conflicts as source ranges (which I'd personally prefer)</div><div>obviously fine as a follow-up patch.</div><div><br>
</div><div>Thanks!</div><div>/Manuel</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  2. Just making sure. When we switch `Replacements` to be an `std::vector` from `std::set` when deduplication happens this new function would be used which is doing something new from before: i.e. conflict detection.<br>

  3. That was the intention.<br>
<br>
  So ready to commit?<br>
<div class="HOEnZb"><div class="h5"><br>
<a href="http://llvm-reviews.chandlerc.com/D1314" target="_blank">http://llvm-reviews.chandlerc.com/D1314</a><br>
<br>
BRANCH<br>
  dedup<br>
<br>
ARCANIST PROJECT<br>
  clang<br>
</div></div></blockquote></div><br></div></div>