[PATCH] D27711: ELF/AArch64: Fix dynamic relocation against local symbol in shared objects
    Rafael Avila de Espindola via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Sun Dec 18 17:56:16 PST 2016
    
    
  
Adhemerval Zanella via Phabricator <reviews at reviews.llvm.org> writes:
> @@ -154,8 +154,8 @@
>        Config->Shared) {
>      if (In<ELFT>::Got->addDynTlsEntry(Body)) {
>        uintX_t Off = In<ELFT>::Got->getGlobalDynOffset(Body);
> -      In<ELFT>::RelaDyn->addReloc(
> -          {Target->TlsDescRel, In<ELFT>::Got, Off, false, &Body, 0});
> +      In<ELFT>::RelaDyn->addReloc({Target->TlsDescRel, In<ELFT>::Got, Off,
> +                                   !Body.isPreemptible(), &Body, 0});
Please use the existing variable. LGTM with that.
Cheers,
Rafael
    
    
More information about the llvm-commits
mailing list