[PATCH] D71086: Let PassBuilder Expose PassInstrumentationCallbacks
Fedor Sergeev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 12 03:23:00 PST 2019
fedor.sergeev added a comment.
Overall I'm fine with this.
Bikeshedding-wise, getPIC seems to be overly cryptic to me.
Do you believe full name getPassInstrumentationCallbacks is too much?
Also, please, add at least one test.
I would start with llvm/unittests/IR/PassBuilderCallbacksTest.cpp
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D71086/new/
https://reviews.llvm.org/D71086
More information about the llvm-commits
mailing list