<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Nov 16, 2016 at 1:19 AM, George Rimar via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">>"local:" is special in that it is not actually part of any version.<br>
>Having two "local: *" will not change the result. I would then suggest<br>
>moving the Locals vector out of VersionDefinition and into<br>
>Configuration.<br>
><br>
>Cheers,<br>
>Rafael<br>
<br>
</span>After some investigations:<br>
1) This change *can* affect on result in next case:<br>
<br>
Version1 {<br>
 global:<br>
  <wildcarded globals 1><br>
local:<br>
 <wildcarded locals 1><br>
}<br>
Version2 {<br>
 global:<br>
  <wildcarded globals 2><br>
local:<br>
 <wildcarded locals 2><br>
}<br>
<br>
If we would have single list of locals, we would process them either before or after lists of globals.<br>
In both cases that is not the same as we had, for example previously order was:<br>
<br>
<wildcarded locals 2>  <wildcarded globals 2>  <wildcarded locals 1>   <wildcarded globals 1><br>
<br>
So after such change it will be one of 2:<br>
<wildcarded locals N>  <wildcarded locals N2 >   <wildcarded globals 2>  <wildcarded globals 1><br>
<wildcarded globals 2>  <wildcarded globals 1> <wildcarded locals N>  <wildcarded locals N2 ><br>
That would be a deviation from GNU behavior.<br></blockquote><div><br></div><div>Does it make real observable difference?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
2) Also this will affect on error reporting. Since when we will have a single list of locals and do not have a maping<br>
local->versionN, it will be problematic to report something like "symbol has version X but was declared as local in version Y",<br>
though we can skip "in version Y" here.<br></blockquote><div><br></div><div>This is not a big issue. You can still emit the same error message by adding a new field or something.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Now I am not sure it worth doing that, do you still in favor or doing such change ?<br>
<br>
Best regards,<br>
<div class="HOEnZb"><div class="h5">George.<br>
______________________________<wbr>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div></div>