[PATCH] D51026: [CodeGen] Implicitly set stackrealign on the main function, if custom stack alignment is used

[Erich Keane via Phabricator via cfe-commits]

erichkeane added inline comments.


================
Comment at: lib/CodeGen/CodeGenFunction.cpp:989
+        CGM.getCodeGenOpts().StackAlignment)
+      Fn->addFnAttr("stackrealign");
+
----------------
Is there not an attribute name for this already in LLVM?  I guess I'm otherwise fine with this, but would want @craig.topper or @rnk to confirm that we're Ok just sending this string as an attribute.


Repository:
  rC Clang

https://reviews.llvm.org/D51026