[PATCH] D134289: [Driver] Make --execute-only the default for aarch64-fuchsia

Alex Brachet via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 20 11:26:32 PDT 2022


abrachet marked an inline comment as done.
abrachet added inline comments.


================
Comment at: clang/lib/Driver/ToolChains/Fuchsia.cpp:93
       CmdArgs.push_back("--fix-cortex-a53-843419");
+    CmdArgs.push_back("--execute-only");
   }
----------------
mcgrathr wrote:
> This is a little confusing to read when it's after the other lines.
> I'd put it first in the block with a blank line after it.
> 
Fixed in commit


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D134289/new/

https://reviews.llvm.org/D134289



More information about the cfe-commits mailing list