<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Mar 6, 2013 at 10:06 AM, Matthew Curtis <span dir="ltr"><<a href="mailto:mcurtis@codeaurora.org" target="_blank">mcurtis@codeaurora.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  
    
  
  <div text="#000000" bgcolor="#FFFFFF"><div class="im">
    <div>On 3/6/2013 11:12 AM, Daniel Dunbar
      wrote:<br>
    </div>
    <blockquote type="cite">
      <div dir="ltr">Hi Matthew,
        <div><br>
        </div>
        <div>Is the motivation just as a cleanup?</div>
      </div>
    </blockquote></div>
    Primarily cleanup, though we have an internal project that involves
    adding a new phase. This makes it a little more straightforward to
    do so.</div></blockquote><div><br></div><div style>Ah, cool.</div><div style> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div text="#000000" bgcolor="#FFFFFF">
<div class="im"><br>
    <blockquote type="cite">
      <div dir="ltr">
        <div><br>
        </div>
        <div>Specific comments on the patch:</div>
        <div><br>
        </div>
        <div>1. getCompilationPhases should take the phase list
          as a reference not a pointer, this is more consistent.</div>
        <div><br>
        </div>
        <div>2. Don't .clear() the list in
          getCompilationPhases.</div>
        <div><br>
        </div>
        <div>3. There aren't enough uses to be worth
          introducing the PhaseList type, just write out the SmallVector
          which makes the code more verbose but more familiar.</div>
        <div><br>
        </div>
      </div>
    </blockquote></div>
    New patch attached incorporating this feedback.<br></div></blockquote><div><br></div><div style>LGTM.</div><div style><br></div><div style> - Daniel</div><div style><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div text="#000000" bgcolor="#FFFFFF">
    <br>
    Thanks for the review!<span class="HOEnZb"><font color="#888888"><br>
    Matthew Curtis</font></span><div class="im"><br>
    <br>
    <blockquote type="cite">
      <div dir="ltr">
        <div> - Daniel</div>
        <div><br>
        </div>
      </div>
      <div class="gmail_extra"><br>
        <br>
        <div class="gmail_quote">On Wed, Mar 6, 2013 at 6:04 AM, Matthew
          Curtis <span dir="ltr"><<a href="mailto:mcurtis@codeaurora.org" target="_blank">mcurtis@codeaurora.org</a>></span>
          wrote:<br>
          <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
            <div text="#000000" bgcolor="#FFFFFF">
              <div>Hello Daniel,<br>
                <br>
                Do you have time to comment on this patch?<br>
                <br>
                Thanks,<br>
                Matthew Curtis.
                <div><br>
                  <br>
                  On 3/1/2013 4:17 PM, Matthew Curtis wrote:<br>
                </div>
              </div>
              <blockquote type="cite">
                <div>There is now a single function to get
                  the list of phases for a given <br>
                  output Type. <br>
                  <br>
                  No functionality change intended. <br>
                  <br>
                  Thanks, <br>
                  Matthew Curtis <br>
                  <br>
                  <br>
                  <fieldset></fieldset>
                  <br>
                </div>
                <pre>_______________________________________________
cfe-commits mailing list
<a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a>
</pre>
              </blockquote>
              <div> <br>
                <br>
                <pre cols="72">-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation</pre>
              </div>
            </div>
          </blockquote>
        </div>
        <br>
      </div>
    </blockquote>
    <br>
    <br>
    <pre cols="72">-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation</pre>
  </div></div>

</blockquote></div><br></div></div>