[lld] [AArch64][GCS][LLD] Introduce -zgcs-report-dynamic Command Line Option (PR #127787)

[Jack Styles via llvm-commits]

================
@@ -1629,7 +1667,6 @@ static void readConfigs(Ctx &ctx, opt::InputArgList &args) {
       std::make_pair("bti-report", &ctx.arg.zBtiReport),
       std::make_pair("cet-report", &ctx.arg.zCetReport),
       std::make_pair("execute-only-report", &ctx.arg.zExecuteOnlyReport),
-      std::make_pair("gcs-report", &ctx.arg.zGcsReport),
       std::make_pair("pauth-report", &ctx.arg.zPauthReport)};
----------------
Stylie777 wrote:

I disagree, I think it is better to handle the options in the location currently being used by this change. Otherwise we will need to parse the users -z options twice, the function currently only does this once for both options.  It will also need a different solution for the inheritance to work, so I prefer to use the function that has been created and utilise this. 

If it makes it clearer for the user, I can add a comment explaining that `-zgcs-report` and `-zgcs-report-dynamic` are both handled inside that function.

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