================
@@ -91,7 +91,9 @@ void fuchsia::Linker::ConstructJob(Compilation &C, const 
JobAction &JA,
     CmdArgs.push_back("--execute-only");
 
     std::string CPU = getCPUName(D, Args, Triple);
-    if (CPU.empty() || CPU == "generic" || CPU == "cortex-a53")
+    if (Args.hasFlag(options::OPT_mfix_cortex_a53_843419,
+                     options::OPT_mno_fix_cortex_a53_843419, true) &&
----------------
bryanpkc wrote:

The GCC docs say that these options are "ignored if an architecture or cpu is 
specified on the command line which does not need the workaround". So I think 
what we have in this PR mostly matches that description.

I could submit a new PR to add a condition to check whether the architecture is 
armv8.1-a or later.

https://github.com/llvm/llvm-project/pull/143915
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to