Skip to content

Use an early IR transform to replace GlobalRefs #311

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vchuravy
Copy link
Member

julia> f() = (X;nothing)
f (generic function with 1 method)

julia> @code_lowered f()
CodeInfo(
1 ─     Main.X
└──     return Main.nothing
)

julia> native_code_typed(f, Tuple{})
static_eval(stmt) = Some([1, 2, 3])
1-element Vector{Any}:
 CodeInfo(
1 ─     return Main.nothing
) => Nothing

@maleadt maleadt force-pushed the vc/rewrite_global_ref branch from 7fde557 to 18470ee Compare May 18, 2022 12:10
@codecov
Copy link

codecov bot commented Jun 1, 2022

Codecov Report

Merging #311 (18470ee) into master (fa386cc) will decrease coverage by 0.51%.
The diff coverage is 53.84%.

@@            Coverage Diff             @@
##           master     #311      +/-   ##
==========================================
- Coverage   85.63%   85.11%   -0.52%     
==========================================
  Files          23       24       +1     
  Lines        2353     2392      +39     
==========================================
+ Hits         2015     2036      +21     
- Misses        338      356      +18     
Impacted Files Coverage Δ
src/GPUCompiler.jl 100.00% <ø> (ø)
src/early_transform.jl 42.85% <42.85%> (ø)
src/jlgen.jl 80.21% <81.81%> (+0.10%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fa386cc...18470ee. Read the comment docs.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant