Agreed, have it just query the Subtarget for now to get it on targets that haven't replaced.<br><br>That said, it'll make it easier if you just call it getObjFileLowering like the current name (which I think is best).<div><br></div><div>Thanks!</div><div><br></div><div>-eric</div><br><div class="gmail_quote">On Wed Nov 05 2014 at 2:23:10 PM Aditya Nandakumar <<a href="mailto:aditya_nandakumar@apple.com">aditya_nandakumar@apple.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Eric,<br>
<br>
Would it be preferable to add a virtual method - getTargetLoweringObjectFile() to TargetMachine ? All targets to could override this.<br>
<br>
Thanks<br>
Aditya<br>
> On Nov 4, 2014, at 2:26 PM, Aditya Nandakumar <<a href="mailto:aditya_nandakumar@apple.com" target="_blank">aditya_nandakumar@apple.com</a>> wrote:<br>
><br>
> Thank you. I’ll update the remaining targets shortly.<br>
><br>
> <a href="http://reviews.llvm.org/D5850" target="_blank">http://reviews.llvm.org/D5850</a><br>
><br>
><br>
<br>
</blockquote></div>