islpy icon indicating copy to clipboard operation
islpy copied to clipboard

Wrong wrapper for `isl_ast_op_type_print_macro`

Open thisiscam opened this issue 4 years ago • 2 comments

I noticed that the ISL function isl_ast_op_type_print_macro got parsed into the wrong class: https://documen.tician.de/islpy/ref_ast.html?highlight=op_type_print_macro#islpy.AstExpr.op_type_print_macro.

I'm leaving an issue here since I'm not sure what's the best approach to fix it.

thisiscam avatar Oct 19 '21 03:10 thisiscam

Hack this logic:

https://github.com/inducer/islpy/blob/a5c7df513b74609c5f7a58ecc20db3c33d506a27/gen_wrap.py#L636-L648

inducer avatar Oct 20 '21 20:10 inducer

Hmm, hacking this naively (similar to line 646) doesn't seem to work because ast_op_type is an enum, and the argument order is reversed compared to ast_op_type_set_print_name .

thisiscam avatar Oct 22 '21 03:10 thisiscam