viroulep updated this revision to Diff 450844.
viroulep added a comment.
I took into account @w2yehia's comment, and I also noticed there was no test in
clang for the `-fpass-plugin` option, so I added one.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D130889/new/
https://reviews.llvm.
viroulep added a comment.
Thanks for your review @w2yehia!
Comment at: llvm/lib/Passes/PassPluginLoader.cpp:23
+void PassPluginLoader::operator=(const std::string &Filename) {
+ sys::SmartScopedLock Lock(*PluginsLock);
+ auto PassPlugin = PassPlugin::Load(Filename);
-
viroulep created this revision.
viroulep added reviewers: w2yehia, mehdi_amini, pcc.
Herald added subscribers: ormris, steven_wu, hiraditya, arichardson, mgorny,
emaste.
Herald added a reviewer: MaskRay.
Herald added a project: All.
viroulep requested review of this revision.
Herald added subscrib