<div dir="ltr">But the relocation type is not architecture dependent, right? The relocation itself knows what the relocation type of itself is. Maybe I can make this member functions of relocation, but adding that predicates to linking context object is not logically correct.</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Feb 13, 2015 at 8:03 AM, Simon Atanasyan <span dir="ltr"><<a href="mailto:simon@atanasyan.com" target="_blank">simon@atanasyan.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I think all target related code should be separated by LinkingContext, TargetHandler etc descendant classes. If moving relocation predicates to the TargetHandler descendants makes the code clear, it is a good idea.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
<a href="http://reviews.llvm.org/D7605" target="_blank">http://reviews.llvm.org/D7605</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</div></div></blockquote></div><br></div>