<div dir="ltr"><br><br><div class="gmail_quote">On Fri, Mar 27, 2015 at 1:14 PM Akira Hatanaka <<a href="mailto:ahatanak@gmail.com">ahatanak@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks.<br>
<br>
I wasn't planning to make the changes I made for AArch64 for other ports that don't use subtarget feature bits in their instprinters, but if it's important to have a uniform interface for the print methods table-gen generates, I can update those ports later and remove PassSubtarget.<br>
<br></blockquote><div><br></div><div>Yeah. More of a long term code cleanliness FIXME than anything else.</div><div><br></div><div>-eric</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
In <a href="http://reviews.llvm.org/D8668#148039" target="_blank">http://reviews.llvm.org/D8668#<u></u>148039</a>, @echristo wrote:<br>
<br>
> Excellent. We're in agreement then.<br>
><br>
> One inline comment and then LGTM.<br>
><br>
> -eric<br>
<br>
<br>
<br>
<br>
<br>
<a href="http://reviews.llvm.org/D8668" target="_blank">http://reviews.llvm.org/D8668</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/<u></u>settings/panel/<u></u>emailpreferences/</a><br>
<br>
<br>
</blockquote></div></div>