Skip to content

Conversation

Roger-luo
Copy link
Collaborator

No description provided.

@codecov
Copy link

codecov bot commented Dec 10, 2021

Codecov Report

Merging #6 (4992b81) into master (33bc3e4) will decrease coverage by 0.97%.
The diff coverage is 85.29%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master       #6      +/-   ##
==========================================
- Coverage   86.41%   85.43%   -0.98%     
==========================================
  Files           8        9       +1     
  Lines         471      515      +44     
==========================================
+ Hits          407      440      +33     
- Misses         64       75      +11     
Impacted Files Coverage Δ
src/patches.jl 70.45% <ø> (ø)
src/interp.jl 66.66% <80.00%> (-19.70%) ⬇️
src/ircode.jl 87.17% <84.21%> (-5.42%) ⬇️
src/passes.jl 81.42% <88.88%> (-0.26%) ⬇️
src/compat.jl 100.00% <100.00%> (ø)
src/utils.jl 83.09% <0.00%> (-0.24%) ⬇️
src/typeinf.jl 100.00% <0.00%> (ø)
src/codeinfo.jl 94.28% <0.00%> (+0.49%) ⬆️
... and 1 more

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 33bc3e4...4992b81. Read the comment docs.

@Roger-luo
Copy link
Collaborator Author

somehow the following case doesn't work anymore

code_ircode(sin, (Float64, ); interp=NativeInterpreter())

@Roger-luo
Copy link
Collaborator Author

@aviatesk mind take a second look if the fix is correct?

@@ -0,0 +1,3 @@
@static if VERSION < v"1.7-"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just drop a support for v1.6...?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure if I can get YaoCompiler working again this weekend, so I think I'll probably try to keep the first version compatible with 1.6 then deprecate it

@@ -48,24 +48,39 @@ Return the `IRCode` object along with inferred return type.
function code_ircode_by_mi(f, mi::MethodInstance; world=get_world_counter(), interp=NativeInterpreter(world))

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

f seems to be dead argument

# passes
ir = f(ir, opt)
opt.src.inferred = true
ir = f(ir, sv)
Copy link
Collaborator Author

@Roger-luo Roger-luo Dec 11, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@aviatesk f is used here, it kinda allow custom IRCode transform from this reflection function since the run_passes function is not called here and is replaced by the default_julia_passes function, so that I can put my own passes using the do ... end syntax to test with

@Roger-luo
Copy link
Collaborator Author

I guess the rest looks good then? I'll merge this first.

@Roger-luo Roger-luo merged commit 0dce551 into master Dec 11, 2021
@Roger-luo Roger-luo deleted the roger/fix-1.7 branch December 11, 2021 17:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants