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

Over-eager unifying of two separate entities in cURL #190

Open
pgoodman opened this issue Sep 30, 2022 · 1 comment
Open

Over-eager unifying of two separate entities in cURL #190

pgoodman opened this issue Sep 30, 2022 · 1 comment
Labels
bug Something isn't working help wanted Extra attention is needed indexer

Comments

@pgoodman
Copy link
Contributor

image
image

@pgoodman
Copy link
Contributor Author

pgoodman commented Oct 7, 2022

I have been mulling over thing. The reason these are merged right now is that the RPC called by ::redelarations goes and tries to do name-based merging, using something the integrates mangled names. I have been thinking that what would be good is an extended mangled name approach, that does something like take into account the lower-level ABI, possibly with LLVM-level info.

So if we have void wait_ms(void); and int wait_ms(int) then we'd have the mangled name as wait_ms, then have a bunch of summaries of return values and parameters. Any integer parameter would be summarized to I, any pointer summarized to P, and void returns summarized to V. So then you'd have: wait_ms:V and wait_ms:II.

@pgoodman pgoodman added bug Something isn't working help wanted Extra attention is needed indexer labels Oct 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed indexer
Projects
None yet
Development

No branches or pull requests

1 participant