Skip to content

Commit 58bc0c4

Browse files
DinoVmeta-codesync[bot]
authored andcommitted
Fix _testinternalcapi build
Summary: After the test interpreter landed our _testinternalcapi build succeeds but is missing a symbol because `interpreter.c` isn't included. To include we need access to CPython's headers in Python and sdt. Once this is fixed we see new failures in `test.test_opcache` because a couple of new tests have trickled in where specialization doesn't work with the JIT. Reviewed By: alexmalyshev Differential Revision: D91243622 fbshipit-source-id: 891e9558adc2d098d688364789602d15f90103d6
1 parent b6e74a8 commit 58bc0c4

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

cinderx/TestScripts/cinder_jit_ignore_tests.txt

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -149,6 +149,8 @@ test.test_dis.DisWithFileTests.test_loop_with_conditional_at_end_is_quickened
149149
test.test_dis.DisWithFileTests.test_super_instructions
150150
test.test_opcache.TestCallCache.test_assign_init_code
151151
test.test_opcache.TestCallCache.test_push_init_frame_fails
152+
test.test_opcache.TestCallCache.test_specialize_call_function_ex_py
153+
test.test_opcache.TestCallCache.test_specialize_call_function_ex_py_fail
152154
test.test_opcache.TestRacesDoNotCrash.*
153155
test.test_opcache.TestSpecializer.*
154156
test.test_type_cache.TypeCacheWithSpecializationTests.test_class_call_specialization_user_type

0 commit comments

Comments
 (0)