<div dir="ltr">(before looking at the code (going off the list of files changed): Generally it's preferred to have tests included with the code being added/changed, where at all possible - is that possible in this case?<br><br>*looks at code* Yeah, that's a lot of code to be adding that looks like it could use a variety of tests. Might even be easier to stage parts of it separately - an iterator & its unit tests at a time, etc. Not sure. Whatever seems to work best. Generally the smaller a patch the exponentially faster the review. (to a point - when the changes are so small they lack context/motivation, that can make the review take longer trying to understand "where it's going")</div><br><div class="gmail_quote"><div dir="ltr">On Sun, Jan 22, 2017 at 10:23 PM Alexis Shaw via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">varno created this revision.<br class="gmail_msg">
<br class="gmail_msg">
In preparation for graph comparison and filtering, this is a library for<br class="gmail_msg">
representing graphs in LLVM. This will enable easier encapsulation and reuse<br class="gmail_msg">
of graphs in llvm-xray.<br class="gmail_msg">
<br class="gmail_msg">
Depends on <a href="https://reviews.llvm.org/D28999" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D28999</a>, <a href="https://reviews.llvm.org/D28225" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D28225</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D29005" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D29005</a><br class="gmail_msg">
<br class="gmail_msg">
Files:<br class="gmail_msg">
  include/llvm/XRay/Graph.h<br class="gmail_msg">
  tools/llvm-xray/xray-graph.cc<br class="gmail_msg">
  tools/llvm-xray/xray-graph.h<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div>