-
Notifications
You must be signed in to change notification settings - Fork 11.1k
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.
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
[PowerPC] Do not string pool globals that are part of llvm used. #66848
[PowerPC] Do not string pool globals that are part of llvm used. #66848
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM; thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@llvm.used = appending global [1 x ptr] [ptr @keep_this], section "llvm.metadata" | ||
@llvm.compiler.used = appending global [1 x ptr] [ptr @keep_this2], section "llvm.metadata" | ||
|
||
declare signext i32 @callee(ptr noundef) local_unnamed_addr |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think local_unnamed_addr
does anything here does it? If not can we remove that and the #0
from blow. Thx
5ae27d2
to
d03571a
Compare
✅ With the latest revision this PR passed the C/C++ code formatter. |
The string pooling pass was incorrectly pooling global varables that were part of llvm.used or llvm.compiler.used. This patch fixes the pass to prevent that by checking each candidate to make sure that it is not in either of those lists.
d03571a
to
0383c03
Compare
Local branch amd-gfx 741a0a2 Merged main:d9962c400f97 into amd-gfx:8bf360256d7b Remote branch main d896b1f [PowerPC] Do not string pool globals that are part of llvm used. (llvm#66848)
The string pooling pass was incorrectly pooling global varables that were part of llvm.used or llvm.compiler.used. This patch fixes the pass to prevent that by checking each candidate to make sure that it is not in either of those lists.