key for storing compiled calls also needs to think about number of parameters (#324)

Author: KristofferC

Project.toml

@@ -19,6 +19,7 @@ Mmap = "a63ad114-7e13-5084-954f-fe012c677804"
 SHA = "ea8e919c-243c-51af-8825-aaa63cd721ce"
 Tensors = "48a634ad-e948-5137-8d70-aa71f2a747f4"
 Test = "8dfed614-e22c-5e08-85e1-65c5234f0b40"
+TableReader = "70df011a-6618-58d7-8e16-3cf9e384cb47"
 
 [targets]
-test = ["Test", "Dates", "Distributed", "Mmap", "SHA", "Tensors"]
+test = ["Test", "Dates", "Distributed", "Mmap", "SHA", "Tensors", "TableReader"]

src/optimize.jl

@@ -287,7 +287,7 @@ function build_compiled_call!(stmt, fcall, code, idx, nargs, sparams, evalmod)
         RetType = RetType[1]
     end
     # When the ccall is dynamic we pass the pointer as an argument so can reuse the function
-    cc_key = (dynamic_ccall ? :ptr : cfunc, RetType, ArgType, evalmod)  # compiled call key
+    cc_key = (dynamic_ccall ? :ptr : cfunc, RetType, ArgType, evalmod, length(sparams))  # compiled call key
     f = get(compiled_calls, cc_key, nothing)
     argnames = Any[Symbol("arg", string(i)) for i = 1:nargs]
     if f === nothing

test/interpret.jl

@@ -562,3 +562,13 @@ g(x) = f(x)
 @test (@interpret g(5)) == g(5)
 f(x) = x*x
 @test (@interpret g(5)) == g(5)
+
+# Regression test https://github.com/JuliaDebug/JuliaInterpreter.jl/issues/300
+module CSVTest
+    using Test
+    using JuliaInterpreter
+    using TableReader
+    const myfile = "smallcsv.csv"
+    @test (@interpret readcsv(myfile)) == readcsv(myfile)
+end
+

test/smallcsv.csv

@@ -0,0 +1,2 @@
+a, b, c
+foo, 1.0, 2.3244342