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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to