================
@@ -402,6 +402,21 @@ static void parseTargetArgs(TargetOptions &opts, 
llvm::opt::ArgList &args) {
   for (const llvm::opt::Arg *currentArg :
        args.filtered(clang::driver::options::OPT_target_feature))
     opts.featuresAsWritten.emplace_back(currentArg->getValue());
+
+  llvm::Triple targetTriple{llvm::Triple(opts.triple)};
+  if (const llvm::opt::Arg *A =
+          args.getLastArg(clang::driver::options::OPT_moutline_atomics,
+                          clang::driver::options::OPT_mno_outline_atomics)) {
+    // Option -moutline-atomics supported for AArch64 target only.
+    if (targetTriple.isAArch64()) {
+      if (A->getOption().matches(
+              clang::driver::options::OPT_moutline_atomics)) {
+        opts.featuresAsWritten.push_back("+outline-atomics");
+      } else {
+        opts.featuresAsWritten.push_back("-outline-atomics");
+      }
+    }
+  }
----------------
kiranchandramohan wrote:

Is this required? I did not see a corresponding change in Clang frontend driver.

https://github.com/llvm/llvm-project/pull/78755
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [flang] [... Mats Petersson via cfe-commits
    • [fla... Kiran Chandramohan via cfe-commits
    • [fla... Mats Petersson via cfe-commits
    • [fla... Kiran Chandramohan via cfe-commits
    • [fla... Mats Petersson via cfe-commits
    • [fla... Mats Petersson via cfe-commits
    • [fla... Valentin Clement バレンタイン クレメン via cfe-commits
    • [fla... Valentin Clement バレンタイン クレメン via cfe-commits

Reply via email to