[PATCH] D30628: [ELF] Handle NONE relocations on AArch64

Rafael Avila de Espindola via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 6 08:02:42 PST 2017


LGTM as is, but bonus if you add a testcase. I think you can do it with
yaml2obj.

Cheers,
Rafael

Petr Hosek via Phabricator via llvm-commits
<llvm-commits at lists.llvm.org> writes:

> phosek created this revision.
> phosek added a project: lld.
> Herald added subscribers: rengolin, aemerson.
>
> Repository:
>   rL LLVM
>
> https://reviews.llvm.org/D30628
>
> Files:
>   ELF/Target.cpp
>
>
> Index: ELF/Target.cpp
> ===================================================================
> --- ELF/Target.cpp
> +++ ELF/Target.cpp
> @@ -1320,6 +1320,8 @@
>    case R_AARCH64_ADR_GOT_PAGE:
>    case R_AARCH64_TLSIE_ADR_GOTTPREL_PAGE21:
>      return R_GOT_PAGE_PC;
> +  case R_AARCH64_NONE:
> +    return R_NONE;
>    }
>  }
>  
>
>
> Index: ELF/Target.cpp
> ===================================================================
> --- ELF/Target.cpp
> +++ ELF/Target.cpp
> @@ -1320,6 +1320,8 @@
>    case R_AARCH64_ADR_GOT_PAGE:
>    case R_AARCH64_TLSIE_ADR_GOTTPREL_PAGE21:
>      return R_GOT_PAGE_PC;
> +  case R_AARCH64_NONE:
> +    return R_NONE;
>    }
>  }
>  
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits


More information about the llvm-commits mailing list