zequanwu added a comment.

In D78659#1997528 <https://reviews.llvm.org/D78659#1997528>, @lebedev.ri wrote:
> 1. Tests missing
> 2. Why isn't `noinline` sufficient, why a whole new attribue?




1. Test added.
2. We want to avoid merging multiple call sites of same function, but 
`noinline` cannot achieve this. For example, we don't want to merge calls in `i 
== 5` and `i == 7` cases in the test case.


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

https://reviews.llvm.org/D78659



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

Reply via email to