Skip to content

cglobal: make distinction of pointer vs name a syntactic distinction#61709

Open
topolarity wants to merge 1 commit intoJuliaLang:masterfrom
topolarity:ct/syntactic-cglobal
Open

cglobal: make distinction of pointer vs name a syntactic distinction#61709
topolarity wants to merge 1 commit intoJuliaLang:masterfrom
topolarity:ct/syntactic-cglobal

Conversation

@topolarity
Copy link
Copy Markdown
Member

@topolarity topolarity commented May 2, 2026

Follow-up to #59165, applying the same semantics to cglobal's first argument.

This PR avoids the route in #61707 and preserves cglobal(x::Ptr, ::Type{T}) semantics. It also does not change how the second-argument (type) is resolved.

The net effect is that the semantics of ccall / cglobal are more similar, and we get to clean up some TODO's and bugs in the compiler that were handling (or not) the exotic non-linearized form of cglobal.

@topolarity topolarity force-pushed the ct/syntactic-cglobal branch 3 times, most recently from a9afc2c to b9a7c2d Compare May 2, 2026 01:50
@topolarity topolarity added the minor change Marginal behavior change acceptable for a minor release label May 2, 2026
@topolarity topolarity force-pushed the ct/syntactic-cglobal branch 4 times, most recently from 51c0ed8 to cccbcd9 Compare May 2, 2026 03:59
@topolarity
Copy link
Copy Markdown
Member Author

@nanosoldier runtests()

1 similar comment
@maleadt
Copy link
Copy Markdown
Member

maleadt commented May 5, 2026

@nanosoldier runtests()

@nanosoldier
Copy link
Copy Markdown
Collaborator

The package evaluation job you requested has completed - possible new issues were detected.
The full report is available.

Report summary

❗ Packages that crashed

11 packages crashed only on the current version.

  • The process was aborted: 1 packages
  • An internal error was encountered: 10 packages

64 packages crashed on the previous version too.

✖ Packages that failed

65 packages failed only on the current version.

  • Package fails to precompile: 2 packages
  • Illegal method overwrites during precompilation: 1 packages
  • Package has test failures: 2 packages
  • There were unidentified errors: 50 packages
  • Test duration exceeded the time limit: 10 packages

1198 packages failed on the previous version too.

✔ Packages that passed tests

15 packages passed tests only on the current version.

  • Other: 15 packages

5743 packages passed tests on the previous version too.

~ Packages that at least loaded

3 packages successfully loaded only on the current version.

  • Other: 3 packages

3396 packages successfully loaded on the previous version too.

➖ Packages that were skipped altogether

375 packages were skipped only on the current version.

  • Package could not be installed: 375 packages

904 packages were skipped on the previous version too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

minor change Marginal behavior change acceptable for a minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants