<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sat, Nov 19, 2016 at 12:44 PM, Simone Atzeni via llvm-dev <span dir="ltr"><<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Thanks Mehdi.<br>
<br>
My pass clones the functions within a module in order to have the original function and an exact copy of the same function but with a different name, i.e. sum() and sum_parallel().<br>
After my pass I will run ThreadSanitizer instrumentation pass only on the new copy of the functions, i.e. only the “_parallel” functions will be instrumented by tsan.<br>
<br>
In some programs that I am compiling, the functions such as atoi and atof get cloned but I want to avoid this, and I noticed that only those functions have GlobalValue::<wbr>AvailableExternallyLinkage, so I was wondering if checking the  linkage is enough to avoid those library functions or there could be situations of functions with that linkage but that have the body implemented in the same module.<br></blockquote><div><br></div><div>Checking for this linkage should be enough. If a function has available_externally linkage, then some other module *must* provide an equivalent, strong definition of that function, so you shouldn't need (or want) to clone such functions.</div><div><br></div><div>The root of your problem is that you clone "atoi" into "atoi_parallel", but you leave the linkage as available_externally, which means that you are also promising that some other module out there provides the symbol "atoi_parallel". Obviously, that isn't the case, so you get linker errors.<br></div><div><br></div><div><div>You could also resolve your problem by giving the parallel clones 'internal' linkage.</div></div></div></div></div>