<div dir="ltr"><div><div><div>Sorry for the typo.<br><br></div>"these are depending on every member of MCAsmInfo"<br><br>should be:<br><br></div>"these are not depending on any member of MCAsmInfo"<br><br>
<br></div>Logan<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Feb 22, 2014 at 10:37 PM, Logan Chien <span dir="ltr"><<a href="mailto:tzuhsiang.chien@gmail.com" target="_blank">tzuhsiang.chien@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 dir="ltr"><div><div><div>Hi Rafael,<br><br></div>I moved these functions to LEB128.h because these are depending on every member of MCAsmInfo and they are more similar with the function such as encodeLEB128().  IMO, it will be better to move them to llvm namespace, so that we don't have to call the functions with MCAsmInfo:: prefix.<br>

<br></div>I have changed these functions to inline functions because the existing functions in LEB128.h are inline functions.  Besides, these functions are small enough to be inlined.  It you have some concern, I can move the function definition to lib/Support/LEB128.cpp.<br>

<br></div>Sincerely,<br>Logan<br></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Feb 22, 2014 at 10:24 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>> --- llvm/trunk/include/llvm/Support/LEB128.h (original)<br>
> +++ llvm/trunk/include/llvm/Support/LEB128.h Sat Feb 22 08:00:39 2014<br>
> @@ -90,6 +90,31 @@ inline uint64_t decodeULEB128(const uint<br>
>    return Value;<br>
>  }<br>
><br>
> +/// Utility function to get the size of the ULEB128-encoded value.<br>
> +inline unsigned getULEB128Size(uint64_t Value) {<br>
> +  unsigned Size = 0;<br>
<br>
</div>Why inline? Why in Support? ARMELFStreamer.cpp would have been able to<br>
use it from MC.<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>