[PATCH][PM] Add pass run listeners to the pass manager.
Duncan P. N. Exon Smith
dexonsmith at apple.com
Tue Apr 29 11:44:04 PDT 2014
On 2014-Apr-28, at 16:29, Chandler Carruth <chandlerc at google.com> wrote:
>
> On Mon, Apr 28, 2014 at 4:17 PM, Reid Kleckner <rnk at google.com> wrote:
> On Mon, Apr 28, 2014 at 4:08 PM, Chandler Carruth <chandlerc at google.com> wrote:
> Second, why are you defining a new callback type? We have std::function now, so I would expect this to be std::function<void(Pass *)>.
>
> We do? http://llvm.org/docs/CodingStandards.html#supported-c-11-language-and-library-features
>
> Arrrg. But I hear someone is working on it.
>
> Another good reason to not bake this stuff into the C API.
I don't think this bakes in the C++-side of the callback type. Unless
I've misread the patch, we can change the storage to a std::function<>
once we're allowed to use them.
We're never going to have std::function<> on the C side.
More information about the llvm-commits
mailing list