<div dir="ltr"><div><div><div>Hi Rafael,<br><br></div>I have changed them to non-inline function in r201939.<br><br></div>I have checked the size of these function on x86_64.  It seems that the size impact are minor.  These function only needs 148 bytes and libLLVMSupport.a has 660379 in its text segment (i.e. the percentage is 0.02224%.)<br>
<br>IMO, I would prefer to put them with encodeSLEB128() because I feel that it is a more straightforward for the developer to include LEB128.h for getSLEB128Size() in comparison with some MC layer header.<br><br></div>Sincerely,<br>
Logan<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Feb 22, 2014 at 10:45 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">> I have changed these functions to inline functions because the existing<br>
> functions in LEB128.h are inline functions.  Besides, these functions are<br>
> small enough to be inlined.  It you have some concern, I can move the<br>
> function definition to lib/Support/LEB128.cpp.<br>
<br>
</div>Please do. Until we know they are hot on some use case it is better to<br>
have them out of line.<br>
<br>
I also worry about Support growing more functionality that is only<br>
used by a small part of LLVM. How about making this functions in the<br>
llvm namespace, but implementing them in MC?<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div>