erichkeane added a comment. Aaron should have final say, but I've included my 2 cents.
================ Comment at: include/clang/Basic/AttrDocs.td:2816 + +Note that x86_64 ABI forces 16-byte stack alignment at the caller side. Taking +it into account 'force_align_arg_pointer' is not needed at x86_64 except rare ---------------- that **the** x86_64 ABI.. "call site" instead of "caller side". ================ Comment at: include/clang/Basic/AttrDocs.td:2817 +Note that x86_64 ABI forces 16-byte stack alignment at the caller side. Taking +it into account 'force_align_arg_pointer' is not needed at x86_64 except rare +cases when caller does not align stack correctly (e.g. flow jumps from i386 ---------------- Instead of "Taking it into account", perhaps, "Because of this," ================ Comment at: include/clang/Basic/AttrDocs.td:2818 +it into account 'force_align_arg_pointer' is not needed at x86_64 except rare +cases when caller does not align stack correctly (e.g. flow jumps from i386 +arch code). ---------------- align **the **stack correctly https://reviews.llvm.org/D37312 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
