Skip to content

Honor --time-region=all-user-plugins for --plugin-config YAML plugins - #1727

Open
Troy Curtiss (tcurtiss) wants to merge 4 commits into
qualcomm:mainfrom
tcurtiss:main_honortiming
Open

Honor --time-region=all-user-plugins for --plugin-config YAML plugins#1727
Troy Curtiss (tcurtiss) wants to merge 4 commits into
qualcomm:mainfrom
tcurtiss:main_honortiming

Conversation

@tcurtiss

Copy link
Copy Markdown

Ensure plugin timing reporting is done for both linker-script PLUGIN_*() and --plugin-config YAML registered plugins

Signed-off-by: Troy Curtiss <trcurtiss@gmail.com>
Signed-off-by: Troy Curtiss <trcurtiss@gmail.com>
@tcurtiss

Copy link
Copy Markdown
Author

Ran clang-format on the branch and updated

Covers Module::readOnePluginConfig()'s three-way OR (bare "Plugin",
scoped plugin name, or all-user-plugins) that gates PrintTimingStats
for plugins registered via a --plugin-config YAML GlobalPlugins entry.

Signed-off-by: Troy Curtiss <trcurtiss@gmail.com>
Signed-off-by: Troy Curtiss <trcurtiss@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants