Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[NewPM] PassBuilder/Clang do not support function merging #47743

Closed
nikic opened this issue Dec 5, 2020 · 1 comment
Closed

[NewPM] PassBuilder/Clang do not support function merging #47743

nikic opened this issue Dec 5, 2020 · 1 comment
Labels
bugzilla Issues migrated from bugzilla llvm:core

Comments

@nikic
Copy link
Contributor

nikic commented Dec 5, 2020

Bugzilla Link 48399
Resolution FIXED
Resolved on Dec 10, 2020 11:49
Version trunk
OS Linux
Blocks #45994
CC @aeubanks

Extended Description

While MergeFunctions supports NPM, it is currently not integrated in PassBuilder and not enabled by clang under -fmerge-functions.

The test https://github.com/llvm/llvm-project/blob/master/clang/test/CodeGenCXX/merge-functions.cpp should have "-fno-experimental-pass-manager" removed once this is fixed.
@aeubanks
Copy link
Contributor

Fixed with https://reviews.llvm.org/D93002

@aeubanks aeubanks mentioned this issue Jul 8, 2020
Closed
@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 10, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bugzilla Issues migrated from bugzilla llvm:core
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@nikic @aeubanks