cinder icon indicating copy to clipboard operation
cinder copied to clipboard

have cinderjit.force_compile() override jit list

Open belm0 opened this issue 4 years ago • 4 comments

I noticed that force_compile() fails if the function isn't in the jit list. Would it be reasonable to change that behavior?

Example use case: I'd like to enable jit on some 3rd party modules using jit list wildcards, but within my own code use decorators and force_compile() to mark functions for compile.

belm0 avatar May 27 '21 23:05 belm0

I was wondering this from #18. Perhaps just doing this and starting with JIT enabled but an empty JIT-list will do what you want. I think we only use force_compile() during tests which generally don't have a JIT list so I'm not sure why this check is present.

jbower-fb avatar Jun 03 '21 20:06 jbower-fb

cc @jbower-fb did you fix this?

tekknolagi avatar Jun 03 '22 02:06 tekknolagi

@tekknolagi, I think you're referring to 4d6a8d562e4eb74f4fadd86a27af8d01d2375cd6 but I'm afraid it does not. There is still a check in _PyJIT_CompileFuncion().

jbower-fb avatar Jun 03 '22 02:06 jbower-fb

I mean there's still a JIT list presence check in _PyJIT_CompileFuncion().

jbower-fb avatar Jun 03 '22 05:06 jbower-fb