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.

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

llvm-upgrade issues bogus warnings about renaming internal symbols #1615

Closed
llvmbot opened this issue Mar 6, 2007 · 2 comments
Closed

llvm-upgrade issues bogus warnings about renaming internal symbols #1615

llvmbot opened this issue Mar 6, 2007 · 2 comments
Labels
bugzilla Issues migrated from bugzilla llvm-tools All llvm tools that do not have corresponding tag quality-of-implementation

Comments

@llvmbot
Copy link
Collaborator

llvmbot commented Mar 6, 2007

Bugzilla Link 1243
Resolution FIXED
Resolved on Mar 06, 2010 14:05
Version trunk
OS All
Reporter LLVM Bugzilla Contributor

Extended Description

While running this test:
test/ExecutionEngine/2003-05-06-LivenessClobber.llx
llvm-upgrade issues this warning:
:9: warning: Renaming function 'test' as 'test.upgrd.1' may cause linkage
errors while reading token '{'.

It really shouldn't as the function is internal and renaming it cannot cause any
harm. Before issuing the warning, a check should be made to see if its internal
and the warning squelched if it is.

@llvmbot
Copy link
Collaborator Author

llvmbot commented Mar 6, 2007

Mine.

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 3, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugzilla Issues migrated from bugzilla llvm-tools All llvm tools that do not have corresponding tag quality-of-implementation
Projects
None yet
Development

No branches or pull requests

1 participant