[PATCH] D49456: [AArch64] Support execute-only LOAD segments.

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 18 10:39:06 PDT 2018


ruiu added inline comments.


================
Comment at: ELF/Writer.cpp:1771
     return Flags | PF_X;
+  if (Config->EMachine == EM_AARCH64 && Config->AArch64ExecuteOnly &&
+      (Flags & PF_X))
----------------
You don't need to check `Config->EMachine == EM_AARCH64` because it's already checked.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D49456





More information about the llvm-commits mailing list