[libcxx-commits] [PATCH] D66614: Add inline to uflow and underflow

[Aditya Kumar via Phabricator via libcxx-commits]

hiraditya added a comment.

In D66614#1657962 <https://reviews.llvm.org/D66614#1657962>, @ldionne wrote:

> Normally, `inline` is used to control the linkage of a function, so that it is ODR-deduplicated across TUs. Correctness of the program is the only thing it should be used for, if we rely on what optimizer folks tell us (e.g. in conferences).


inline hint is used to determine inlining threshold, so it does have effect on llvm's inlining decision. e.g., https://github.com/llvm-mirror/llvm/blob/master/lib/Analysis/InlineCost.cpp#L917

> For that reason, my first reaction would be to consider this failed optimization an optimizer bug, not a library bug.




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

https://reviews.llvm.org/D66614