<div dir="ltr">Thanks, the patches got committed in: <span style="line-height:normal">r244685, r244686, r244687, r244693 and r244698.</span><div><br></div><div>Alex</div></div><div class="gmail_extra"><br><div class="gmail_quote">2015-08-07 13:09 GMT-07:00 Akira Hatanaka <span dir="ltr"><<a href="mailto:ahatanak@gmail.com" target="_blank">ahatanak@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Alex,<div><br></div><div>The patch looks fine to me.<div><div class="h5"><div><br></div><div><div class="gmail_extra"><div class="gmail_quote">On Thu, Aug 6, 2015 at 3:11 PM, Alex L <span dir="ltr"><<a href="mailto:arphaman@gmail.com" target="_blank">arphaman@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi,<div><br></div><div>There are 5 patches attached below:</div><div><br></div><div>- The first one runs clang-format on PseudoSourceValue.cpp and PseudoSourceValue.h</div><div><br></div><div>- The second one fixes some documentation comments and variable names in</div><div>   <span style="line-height:normal">PseudoSourceValue.cpp and PseudoSourceValue.h.</span></div><div><span style="line-height:normal"><br></span></div><div><span style="line-height:normal">- The third patch introduces a new PSVKind enum in PseudoSourceValue class, which is used</span></div><div><span style="line-height:normal">   to distinguish between the various types of pseudo source values.</span></div><div><span style="line-height:normal"><br></span></div><div><span style="line-height:normal">-  The fourth patch </span><span style="line-height:normal">removes the global manager variable which is responsible for storing</span></div><div>   and allocating pseudo source values and instead it uses a new manager class which</div><div>   is owned <span style="line-height:normal">by the machine function. </span></div><div><span style="line-height:normal"><br></span></div><div><span style="line-height:normal">-  The fifth patch converts the MipsCallEntry PseudoSourceValue subclass into two</span></div><div><span style="line-height:normal">    target neutral PseudoSourceValue subclasses: </span>GlobalValuePseudoSourceValue</div><div>    and ExternalSymbolPseudoSourceValue.</div><div><br></div><div>These changes will make it easier to serialize pseudo source values by getting rid</div><div>of all <span style="line-height:normal">the target specific pseudo source value subclasses in tree.</span></div><div><br></div><div>Cheers,</div><div>Alex</div></div>
</blockquote></div><br></div></div></div></div></div></div>
</blockquote></div><br></div>