<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Aug 6, 2015, at 3:21 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com" class="">dblaikie@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class="">No need for "LTParams(), " in the MCAssembler ctor, is there?<br class=""></div></div></blockquote><div><br class=""></div><div>'course not.</div><br class=""><blockquote type="cite" class=""><div class=""><div dir="ltr" class="">Is the '+' needed here "+Params.DWARF2LineOpcodeBase - 1” ?<br class=""></div></div></blockquote><div><br class=""></div><div>:-) I must have copied over a failed hunk from a .rej file and then clang-format-diff stuck it to the expression.</div><br class=""><blockquote type="cite" class=""><div class=""><div dir="ltr" class="">There's some unrelated formatting changes (forceExpAbs?) that could be removed.<br class=""><br class="">& I guess this has been run through clang-format?</div></div></blockquote><div><br class=""></div><div>Hence the formatting changes. </div><div>New patch attached.</div><div><br class=""></div><div></div></div></body></html>