[PATCH] D130889: [llvm] Introduce a pass plugin registry and loader

2022-08-08 Thread Philippe Virouleau via Phabricator via cfe-commits
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.

[PATCH] D130889: [llvm] Introduce a pass plugin registry and loader

2022-08-08 Thread Philippe Virouleau via Phabricator via cfe-commits
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); -

[PATCH] D130889: [llvm] Introduce a pass plugin registry and loader

2022-08-01 Thread Philippe Virouleau via Phabricator via cfe-commits
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