<div dir="ltr"><div>Hi Joey,</div><div><br></div>On Thu, Sep 12, 2013 at 3:09 AM, Joey Gouly <span dir="ltr"><<a href="mailto:joey.gouly@arm.com" target="_blank">joey.gouly@arm.com</a>></span> wrote:<br><div class="gmail_extra">

<div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Rui,<br>
<div class="im"><br>
> +  /// Returns the decorated name of the given symbol name. On 32-bit x86,<br>
it<br>
> +  /// adds "_" at the beginning of the string. On other architectures,<br>
the<br>
> +  /// return value is the same as the argument.<br>
> +  StringRef decorateSymbol(StringRef name) const {<br>
> +    // Because we don't support architectures other than 32-bit x86,<br>
we'll<br>
> +    // prepend an underscore unconditionally.<br>
> +    std::string str = "_";<br>
> +    str.append(name);<br>
> +    return allocateString(str);<br>
> +  }<br>
> +<br>
<br>
</div>The doxygen comment and the implementation don't match here. I know you said<br>
we only support<br>
32-bit x86 for now, but how hard would it be to implement it correctly now?<br>
Or change the doxygen comment to match the current implementation?<br></blockquote><div><br></div><div>The problem is that we don't have a variable indicating which platform we are linking to, so we can't say wether we are producing x86-32 binary or not. We assume we are always creating x86-32 binary instead in many places... That being said, it should be easy to add such variable and use it here. Let me do that.</div>

</div></div></div>