[lld] [llvm] [LLD][ELF][AArch64] Add support for SHF_AARCH64_PURECODE ELF section flag (3/3) (PR #125689)

[Peter Smith via llvm-commits]

=?utf-8?q?Csanád_Hajdú?= <csanad.hajdu at arm.com>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/125689 at github.com>


================
@@ -459,6 +459,8 @@ static void checkOptions(Ctx &ctx) {
       ErrAlways(ctx) << "-z gcs-report only supported on AArch64";
     if (ctx.arg.zGcs != GcsPolicy::Implicit)
       ErrAlways(ctx) << "-z gcs only supported on AArch64";
+    if (ctx.arg.zExecuteOnlyReport != "none")
----------------
smithp35 wrote:

When I first saw the PR, my immediate thought was to separate -z execute-only-report into a separate patch and implement for Arm/AArch64. This may be worth doing as you can write tests for that in that patch. Will also help speed up the LLD review for this part as there's less code to look at. 

https://github.com/llvm/llvm-project/pull/125689