================
@@ -1982,7 +1982,7 @@ unsigned tools::ParseFunctionAlignment(const ToolChain 
&TC,
     return 0;
 
   if (A->getOption().matches(options::OPT_falign_functions))
-    return 0;
+    return 4; // log2(16)
----------------
pogo59 wrote:

Ah, I should have said "the commentary" on ParseFunctionAlignment, which does 
say that no argument means 16.
Which documentation are you quoting? I don't see this option described in [the 
clang user manual](https://clang.llvm.org/docs/UsersManual.html).

I'm not aware that the driver, or clang generally, knows about preferred 
alignment by target, but I guess we can invent a parameter for that.

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

Reply via email to