[PATCH] D33515: Force RegisterStandardPasses to construct std::function in the IPO library.

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 14 14:28:08 PDT 2017


mehdi_amini added a comment.

In https://reviews.llvm.org/D33515#780599, @marsupial wrote:

> > I'm missing the *why* using a "real function" (I guess you meant "function pointer") matter?
>
> Don't understand this.
>  Is the comment really confusing/unhelpful?


Yes. First "real" doesn't seem accurate. I'm unsure what it means.
Then I still have no idea why this patch is needed. I don't see any explanation anywhere.


https://reviews.llvm.org/D33515





More information about the llvm-commits mailing list