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

Jump to bottom

JIT should not codegen available_externally functions. #6107

Closed
llvmbot opened this issue Dec 9, 2009 · 7 comments
Closed

JIT should not codegen available_externally functions. #6107

llvmbot opened this issue Dec 9, 2009 · 7 comments
Labels
bugzilla Issues migrated from bugzilla mcjit

Comments

@llvmbot
Copy link
Collaborator

llvmbot commented Dec 9, 2009

Bugzilla Link 5735
Resolution FIXED
Resolved on Jan 27, 2010 15:50
Version trunk
OS All
Depends On llvm/llvm-bugzilla-archive#5737
Attachments Failing test + assertion, Proposed fix
Reporter LLVM Bugzilla Contributor
CC @nlewycky

Extended Description

The attached patch currently fails because the JIT tries to compile the available_externally function instead of dlsym'ing the existing definition.
@llvmbot
Copy link
Collaborator Author

llvmbot commented Dec 17, 2009

Fixed by r91626.

@llvmbot
Copy link
Collaborator Author

llvmbot commented Dec 23, 2009

I reverted part of this fix in r91943 because it broke VMKit. Lazy-loaded available_externally functions are broken again.

@llvmbot
Copy link
Collaborator Author

llvmbot commented Dec 23, 2009

Test for ghost available_externally functions

@llvmbot
Copy link
Collaborator Author

llvmbot commented Jan 27, 2010

Fixed in r94686.

@llvmbot
Copy link
Collaborator Author

llvmbot commented Nov 27, 2021

mentioned in issue llvm/llvm-bugzilla-archive#5737

1 similar comment
@llvmbot
Copy link
Collaborator Author

llvmbot commented Nov 27, 2021

mentioned in issue llvm/llvm-bugzilla-archive#5737

@llvmbot
Copy link
Collaborator Author

llvmbot commented Nov 27, 2021

mentioned in issue llvm/llvm-bugzilla-archive#5746

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 3, 2021
augusto2112 pushed a commit to augusto2112/llvm-project that referenced this issue Feb 2, 2023
@jansvoboda11
Merge pull request llvm#6107 from apple/jan_svoboda/stable-20220421-r…
75d33a8 
…emove-new-scanner-API

rdar://104607183 [libclang] Remove the _v4 and _v5 scanning APIs
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 mcjit
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@llvmbot