[PATCH] D68410: [AttrDocs] document always_inline

Fri Dec 4 15:36:45 PST 2020

nickdesaulniers added a comment.

Sorry, I should not have waiting this long to update this patch...

================
Comment at: clang/include/clang/Basic/AttrDocs.td:4401
+disabled. Does not guarantee that inline substitution actually occurs.
+}];
+  let Heading = "always_inline";
----------------
lebedev.ri wrote:
> jdoerfert wrote:
> > It is more than that. This would imply that with optimizations enabled there is no effect. I would mention that the inline heuristic is disabled and inlining is always attempted, w/ or w/o optimizations.
> This comment wasn't addressed.
I don't follow. I'm not sure if this comment has bit rot?  I think it's critical to mention that `always_inline` does not mean always, as inline substitution MAY fail.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68410