<div dir="ltr">The patches should apply cleanly on r<span style="color:rgb(0,0,0);font-family:Helvetica;font-size:13px">244240.</span></div><div class="gmail_extra"><br><div class="gmail_quote">2015-08-06 15:11 GMT-07:00 Alex L <span dir="ltr"><<a href="mailto:arphaman@gmail.com" target="_blank">arphaman@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,<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>