[issue44501] Packing constant call arguments

2022-04-06 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- resolution: -> rejected stage: patch review -> resolved status: open -> closed ___ Python tracker <https://bugs.python.or

[issue40001] ignore errors in SimpleCookie

2022-03-14 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- nosy: -BTaskaya ___ Python tracker <https://bugs.python.org/issue40001> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue40001] ignore errors in SimpleCookie

2022-03-14 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- pull_requests: -29968 ___ Python tracker <https://bugs.python.org/issue40001> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue40001] ignore errors in SimpleCookie

2022-03-14 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- nosy: +BTaskaya nosy_count: 3.0 -> 4.0 pull_requests: +29968 pull_request: https://github.com/python/cpython/pull/19214 ___ Python tracker <https://bugs.python.org/issu

[issue29418] inspect.isroutine does not return True for some bound builtin methods

2022-02-17 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: New changeset 02815d939362d4093a95da650e7fbddabf147eb0 by Hakan Çelik in branch 'main': bpo-29418: Add inspect.ismethodwrapper to whatsnew (GH-31377) https://github.com/python/cpython/commit/02815d939362d4093a95da650e7fbddabf147eb0 -- nosy

[issue29418] inspect.isroutine does not return True for some bound builtin methods

2022-02-16 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker <https://bugs.python.or

[issue46725] Unpacking without parentheses is allowed since 3.9

2022-02-11 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: Since this was already allowed in 3.9 and 3.10 stable relases, and since it is consistent with the RHS of an assignment (something = *a, *b); I'd lean towards keeping it (and maybe fixing the old parser's grammar to reflect that) and documenting

[issue46719] Call not visited in ast.NodeTransformer

2022-02-11 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: > It looks like calls in function arguments are ignored. Please share a small, self-contained reproducer. >From what I can assume by this message, the problem is that you are not >calling self.generic_visit(node) on the first call you are handlin

[issue46674] Two if in a row in generators

2022-02-07 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: As Serhiy mentioned, this is a valid syntax. -- nosy: +BTaskaya resolution: -> not a bug stage: -> resolved status: open -> closed ___ Python tracker <https://bugs.python.or

[issue46520] `ast.unparse` produces syntactically illegal code for identifiers that look like reserved words

2022-01-26 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: Technically, this is a bug on the fact that it breaks the only guarantee of ast.unparse: > Unparse an ast.AST object and generate a string with code that would produce > an equivalent ast.AST object if parsed back with ast.parse(). But I am not

[issue46422] Why do we need `dis.Positions`?

2022-01-24 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: New changeset 58f3d980989c7346ad792d464c1d749dcec6af63 by Nikita Sobolev in branch 'main': bpo-46422: use `dis.Positions` in `dis.Instruction` (GH-30716) https://github.com/python/cpython/commit/58f3d980989c7346ad792d464c1d749dcec6af63

[issue46483] `pathlib.PurePath.__class_getitem__` does not return `GenericAlias`

2022-01-23 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: New changeset 1f715d5bd3bc9ff444e109b6bbd13011913681b1 by Nikita Sobolev in branch 'main': bpo-46483: change `PurePath.__class_getitem__` to return `GenericAlias` (GH-30822) https://github.com/python/cpython/commit/1f715d5bd3bc9ff444e109b6bbd13011913681b1

[issue46212] Avoid temporary `varargs` tuple creation in argument passing

2022-01-21 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: > Note that _PyArg_UnpackKeywordsWithVararg is defined with PyAPI_FUNC. > Changing its argument spec is strictly a backwards incompatible change, IIUC. AFAIK we have committed _PyArg_UnpackKeywordsWithVararg on 3.11 alpha, so I think it should b

[issue40729] Update the list of auto-generated files in .gitattributes

2022-01-21 Thread Batuhan Taskaya
New submission from Batuhan Taskaya : Superseeded by https://github.com/python/core-workflow/issues/425 -- resolution: -> later stage: patch review -> resolved status: open -> closed ___ Python tracker <https://bugs.python.or

[issue46441] Caret points to wrong line on 'return yield 42' in REPL

2022-01-20 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: New changeset 30fb6d073d9ca00dff8e4155c523cdfa63abab6b by Batuhan Taskaya in branch 'main': bpo-46441: Add a boilerplate to test syntax errors in interactive mode (GH-30720) https://github.com/python/cpython/commit/30fb6d073d9ca00dff8e4155c523cdfa63abab6b

[issue46441] Caret points to wrong line on 'return yield 42' in REPL

2022-01-20 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- nosy: +BTaskaya nosy_count: 3.0 -> 4.0 pull_requests: +28911 pull_request: https://github.com/python/cpython/pull/30720 ___ Python tracker <https://bugs.python.org/issu

[issue46422] Why do we need `dis.Positions`?

2022-01-19 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: The initial aim of the dis.Positions was to provide an interface like AST nodes. So you could do for instr in dis.Bytecode(source): print("located in: ", instr.positions.lineno) instead of for instr in dis.Bytecode(source): if instr

[issue46289] AST: FomattedValue conversion's default value should be -1

2022-01-07 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- stage: patch review -> resolved status: open -> closed versions: +Python 3.10, Python 3.11 -Python 3.9 ___ Python tracker <https://bugs.python.org/i

[issue46289] AST: FomattedValue conversion's default value should be -1

2022-01-07 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: Should we backport this? On Sat, Jan 8, 2022, 12:05 AM miss-islington wrote: > > miss-islington added the > comment: > > > New changeset d382f7ee0b98e4ab6ade9384268f25c06be462ad by Batuhan Taskaya > in branch 'main': > bp

[issue46289] AST: FomattedValue conversion's default value should be -1

2022-01-07 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- keywords: +patch pull_requests: +28670 stage: -> patch review pull_request: https://github.com/python/cpython/pull/30467 ___ Python tracker <https://bugs.python.org/issu

[issue46289] AST: FomattedValue conversion's default value should be -1

2022-01-07 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: ASDL technically allows it to be None though neither compiler nor ast.unparse can work with it at this moment. > FormattedValue(expr value, int? conversion, expr? format_spec) https://github.com/python/cpython/blob/b127e70a8a682fe869c22ce04c379bd85a00d

[issue46073] ast.unparse produces: 'FunctionDef' object has no attribute 'lineno' for valid module

2021-12-14 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: >From the first glance, this seems like a duplicate of bpo-44896. -- nosy: +BTaskaya ___ Python tracker <https://bugs.python.org/issu

[issue44896] AttributeError in ast.unparse

2021-12-14 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- status: open -> closed ___ Python tracker <https://bugs.python.org/issue44896> ___ ___ Python-bugs-list mailing list Unsubscrib

[issue45974] Using walrus produces different/unoptimized bytecode

2021-12-03 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: See: https://bugs.python.org/issue42282 and the reason for not backporting this: https://bugs.python.org/issue42282#msg380506 -- components: +Interpreter Core -Parser nosy: +BTaskaya resolution: -> duplicate stage: -> resolved status

[issue45907] Optimize literal comparisons and contains

2021-11-27 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: We just rejected the same issue 2 days ago. If you feel very strong about this; instead of creating new ticket in the same place, you might want to try python-dev instead. Re: the todo comment, feel free to send a patch that removes it. I don't thank

[issue45843] Optimizing LOAD_CONST followed by COMPARE_OP (or IS_OP)

2021-11-20 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: > Result of `255581293 > 12938373 and 113314 < 2`: Is there a similar pattern that happens a lot on the client code? The AST optimizer currently only optimizes common cases. -- nosy: +BTaskaya, pablogsal, serhiy.

[issue45669] An 'ascii_alphanumerics' variable is missing in the 'strings' lib

2021-10-29 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: I've found 81 occurrences of this pattern among 52 projects on a dataset of top PyPI packages (~3.5K). So I'd say +1 on including this. -- nosy: +BTaskaya ___ Python tracker <https://bugs.python.org/issue45

[issue45408] [fuzzer] Weird input with continuation and newlines causes null deref in parser

2021-10-07 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- nosy: +BTaskaya ___ Python tracker <https://bugs.python.org/issue45408> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue45017] move opcode-related logic from modulefinder to dis

2021-09-09 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: Does find_imports/find_store_names have to be public? I think the relocation is fine, but making them public without any use case from outside can be avoided at least for now. -- nosy: +BTaskaya ___ Python

[issue43950] Include column offsets for bytecode instructions

2021-09-04 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- pull_requests: +26588 pull_request: https://github.com/python/cpython/pull/28150 ___ Python tracker <https://bugs.python.org/issue43

[issue43950] Include column offsets for bytecode instructions

2021-09-03 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: New changeset 85ea2d6165dec0cffa6302eb6dc40406eae1edf5 by Batuhan Taskaya in branch 'main': bpo-43950: support positions for dis.Instructions created through dis.Bytecode (GH-28142) https://github.com/python/cpython/commit

[issue43950] Include column offsets for bytecode instructions

2021-09-03 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- pull_requests: +26581 pull_request: https://github.com/python/cpython/pull/28142 ___ Python tracker <https://bugs.python.org/issue43

[issue44896] AttributeError in ast.unparse

2021-08-14 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- keywords: +patch pull_requests: +26245 stage: -> patch review pull_request: https://github.com/python/cpython/pull/27770 ___ Python tracker <https://bugs.python.org/issu

[issue44896] AttributeError in ast.unparse

2021-08-14 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: I don't think this is really an issue considering some other functionalities that consumes AST code (e.g `compile`) will expect it to be annotated with location metadata. This is why the `ast` module already comes bundled with a utility function called

[issue42837] Symbol table incorrectly identifies code as a generator, when 'yield' occurs in an annotation

2021-08-13 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: AFAIK this is already resolved on the main (https://bugs.python.org/issue42725) -- ___ Python tracker <https://bugs.python.org/issue42

[issue44880] Document code.replace()

2021-08-10 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: See: https://github.com/python/cpython/pull/17776 -- nosy: +BTaskaya ___ Python tracker <https://bugs.python.org/issue44

[issue44825] node.annotation is not a str in `ast`'s `class _Unparser(NodeVisitor)`

2021-08-04 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: > Which for some reason has made the `annotation` a `str` rather than `Name`. > So I think it's my bug then? Yes, from what I can see it is a bug in your code. > body=[AnnAssign(annotation='int', The 'annotation' field in 'AnnAssi

[issue44825] node.annotation is not a str in `ast`'s `class _Unparser(NodeVisitor)`

2021-08-04 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: @samuelmarks can you share the full reproducer / AST and what you expect it to be unparsed? (you can use gist or other pasting services if it is too big) -- ___ Python tracker <https://bugs.python.org/issue44

[issue44825] node.annotation is not a str in `ast`'s `class _Unparser(NodeVisitor)`

2021-08-04 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- nosy: +BTaskaya ___ Python tracker <https://bugs.python.org/issue44825> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue43950] Include column offsets for bytecode instructions

2021-07-24 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: There is a subtle difference between traceback.py and Python/traceback.c which makes the latter (C version) output the line without trimming the trailing whitespace. The Python version simply uses `.strip()` which strips both the left (starting) and right

[issue43950] Include column offsets for bytecode instructions

2021-07-24 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- pull_requests: +25882 pull_request: https://github.com/python/cpython/pull/27339 ___ Python tracker <https://bugs.python.org/issue43

[issue43950] Include column offsets for bytecode instructions

2021-07-24 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: New changeset 4f5980a4f57dab68b9137304f58bd08891d43d5a by Batuhan Taskaya in branch 'main': bpo-43950: support long lines in traceback.py (GH-27336) https://github.com/python/cpython/commit/4f5980a4f57dab68b9137304f58bd08891d43d5a

[issue43950] Include column offsets for bytecode instructions

2021-07-24 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: New changeset ef8b8535cb3cd705392af9b927d6ff336d98427d by Batuhan Taskaya in branch 'main': bpo-43950: check against the raw string, not the pyobject (GH-27337) https://github.com/python/cpython/commit/ef8b8535cb3cd705392af9b927d6ff336d98427d

[issue43950] Include column offsets for bytecode instructions

2021-07-24 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- pull_requests: +25880 pull_request: https://github.com/python/cpython/pull/27337 ___ Python tracker <https://bugs.python.org/issue43

[issue43950] Include column offsets for bytecode instructions

2021-07-24 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- pull_requests: +25879 pull_request: https://github.com/python/cpython/pull/27336 ___ Python tracker <https://bugs.python.org/issue43

[issue43950] Include column offsets for bytecode instructions

2021-07-23 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- pull_requests: +25858 pull_request: https://github.com/python/cpython/pull/27313 ___ Python tracker <https://bugs.python.org/issue43

[issue42355] symtable: get_namespace doesn't check whether if there are multiple namespaces or no namespaces at all

2021-07-18 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- type: security -> behavior ___ Python tracker <https://bugs.python.org/issue42355> ___ ___ Python-bugs-list mailing list Un

[issue42355] symtable: get_namespace doesn't check whether if there are multiple namespaces or no namespaces at all

2021-07-18 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker <https://bugs.python.or

[issue42355] symtable: get_namespace doesn't check whether if there are multiple namespaces or no namespaces at all

2021-07-18 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: New changeset a045991f60b51636a784623dda7ad84b5b2c6b73 by Batuhan Taskaya in branch 'main': bpo-42355: symtable.get_namespace() now checks whether there are multiple or any namespaces found (GH-23278) https://github.com/python/cpython/commit

[issue43950] Include column offsets for bytecode instructions

2021-07-17 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- pull_requests: +25758 pull_request: https://github.com/python/cpython/pull/27217 ___ Python tracker <https://bugs.python.org/issue43

[issue20291] Argument Clinic should understand *args and **kwargs parameters

2021-07-17 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: New changeset f88e138a1aa3b9a9e013963e4fd7d5cce6a0b85c by Ken Jin in branch 'main': bpo-20291: Fix MSVC warnings in getargs.c (GH-27211) https://github.com/python/cpython/commit/f88e138a1aa3b9a9e013963e4fd7d5cce6a0b85c

[issue20201] Argument Clinic: rwbuffer support broken

2021-07-16 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- Removed message: https://bugs.python.org/msg397638 ___ Python tracker <https://bugs.python.org/issue20201> ___ ___ Python-bug

[issue20201] Argument Clinic: rwbuffer support broken

2021-07-16 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: New changeset 9af34c935185eca497617a216d141c72ffaeae9c by Batuhan Taskaya in branch 'main': bpo-20201: variadic arguments support for AC (GH-18609) https://github.com/python/cpython/commit/9af34c935185eca497617a216d141c72ffaeae9c -- nosy

[issue43950] Include column offsets for bytecode instructions

2021-07-13 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- pull_requests: +25669 pull_request: https://github.com/python/cpython/pull/27126 ___ Python tracker <https://bugs.python.org/issue43

[issue43950] Include column offsets for bytecode instructions

2021-07-13 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- pull_requests: +25664 pull_request: https://github.com/python/cpython/pull/27117 ___ Python tracker <https://bugs.python.org/issue43

[issue43897] Implement support for validation of pattern matching ASTs

2021-07-08 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- pull_requests: +25624 pull_request: https://github.com/python/cpython/pull/27074 ___ Python tracker <https://bugs.python.org/issue43

[issue43950] Include column offsets for bytecode instructions

2021-07-05 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- pull_requests: +25597 pull_request: https://github.com/python/cpython/pull/27037 ___ Python tracker <https://bugs.python.org/issue43

[issue43950] Include column offsets for bytecode instructions

2021-07-04 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- pull_requests: +25574 pull_request: https://github.com/python/cpython/pull/27015 ___ Python tracker <https://bugs.python.org/issue43

[issue43950] Include column offsets for bytecode instructions

2021-07-04 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- pull_requests: +25571 pull_request: https://github.com/python/cpython/pull/27011 ___ Python tracker <https://bugs.python.org/issue43

[issue44501] Packing constant call arguments

2021-07-03 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: > BTW, why do you not fold 2-argument calls? Seems like it is making stuff worse. > I suspect that in most of other examples the function execution time is too > large to make the optimization meaningful. This is (to some extent) a constant fol

[issue44501] Packing constant call arguments

2021-07-03 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: > I am still not sure that it is worth to add 50 lines of the C code to > optimize 0.7% of calls. As I stated, the reason that this is 'relatively' lower is that there are just too many calls, so it just allocates a rather small part of the tot

[issue43950] Include column offsets for bytecode instructions

2021-07-02 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- pull_requests: +25557 pull_request: https://github.com/python/cpython/pull/26997 ___ Python tracker <https://bugs.python.org/issue43

[issue44313] Generate LOAD_ATTR+CALL_FUNCTION instead of LOAD_METHOD+CALL_METHOD for imports

2021-07-01 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: New changeset 0d7f61ddb074659d8c18c8f5ac86a6a18e41f9e5 by Batuhan Taskaya in branch 'main': bpo-44313: bump up magic (#26983) https://github.com/python/cpython/commit/0d7f61ddb074659d8c18c8f5ac86a6a18e41f9e5

[issue44313] Generate LOAD_ATTR+CALL_FUNCTION instead of LOAD_METHOD+CALL_METHOD for imports

2021-07-01 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- pull_requests: +25545 pull_request: https://github.com/python/cpython/pull/26983 ___ Python tracker <https://bugs.python.org/issue44

[issue44313] Generate LOAD_ATTR+CALL_FUNCTION instead of LOAD_METHOD+CALL_METHOD for imports

2021-07-01 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker <https://bugs.python.or

[issue40528] Improve / Clear ASDL generator

2021-06-27 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: These were all the cleanups on my checklist, so can close the issue now. I'll probably create other issues if something additional comes up -- resolution: -> fixed stage: patch review -> resolved status: open -&g

[issue40528] Improve / Clear ASDL generator

2021-06-27 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: New changeset 107a2c59c91b3911bdd6dfdb83271c588c506a5a by Batuhan Taskaya in branch 'main': bpo-40528: fix is_simple(sum)s behavior for attributes (GH-26918) https://github.com/python/cpython/commit/107a2c59c91b3911bdd6dfdb83271c588c506a5a

[issue40528] Improve / Clear ASDL generator

2021-06-26 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- pull_requests: +25491 pull_request: https://github.com/python/cpython/pull/26918 ___ Python tracker <https://bugs.python.org/issue40

[issue44505] loop invariant code motion

2021-06-24 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- resolution: -> not a bug stage: -> resolved status: open -> closed ___ Python tracker <https://bugs.python.or

[issue44505] loop invariant code motion

2021-06-24 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: There are multiple issues with doing 'static' loop invariant code motion in python, but the most obvious one is how would you detect if anything in the body is related to the loop or not? The trivial way that the compiled languages do this optimization

[issue44501] Packing constant call arguments

2021-06-24 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: I've tested this with pyperformance, and no significant (1.02 faster on some, 1.01x slower on others, nothing significant) change on those so this is a targeted optimization. -- ___ Python tracker <ht

[issue44501] Packing constant call arguments

2021-06-24 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- keywords: +patch pull_requests: +25477 stage: -> patch review pull_request: https://github.com/python/cpython/pull/26901 ___ Python tracker <https://bugs.python.org/issu

[issue40528] Improve / Clear ASDL generator

2021-06-24 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: New changeset 6c76df2b86d742cc294beae7f9b6bafabb946ad5 by Batuhan Taskaya in branch 'main': bpo-40528: move asdl identifier collection to the new metadata system (GH-26858) https://github.com/python/cpython/commit/6c76df2b86d742cc294beae7f9b6bafabb946ad5

[issue44501] Packing constant call arguments

2021-06-23 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: Here is my branch for the reference (I didn't worked on about it much, so still lacks some error handling etc); https://github.com/isidentical/cpython/blob/b556d1172b08c65b88093f7ff1dadc985ce72f62/Python/ast_opt.c#L634-L698

[issue44501] Packing constant call arguments

2021-06-23 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: > I already experimented with this in issue33325. It does not worth an effort. Yea, that is what the results for the CFG optimization shows to me (1.13x at best, ~1.03x in real cases). > For optimizing calls with constant arguments, it look

[issue44501] Packing constant call arguments

2021-06-23 Thread Batuhan Taskaya
New submission from Batuhan Taskaya : It is a common scenario to make calls with only constant arguments (e.g to datetime.datetime/os.path.join/re.match.group/nox.session.run etc) and the bytecode that we currently generate looks like this; f(1,2,3,4,5,6) 1 0 LOAD_NAME

[issue40528] Improve / Clear ASDL generator

2021-06-22 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- pull_requests: +25435 pull_request: https://github.com/python/cpython/pull/26858 ___ Python tracker <https://bugs.python.org/issue40

[issue40528] Improve / Clear ASDL generator

2021-06-22 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: New changeset 35ad425866d591c33d7f2be2b9da8bce2bff9523 by Batuhan Taskaya in branch 'main': bpo-40528: Implement a metadata system for ASDL Generator (GH-20193) https://github.com/python/cpython/commit/35ad425866d591c33d7f2be2b9da8bce2bff9523

[issue44369] Improve syntax error for wrongly closed strings

2021-06-12 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: >From what I can share by my experience, this error happens soo much. So I'd >love to see it. For a more consistent message with others, maybe it could be something like this; SyntaxError: invalid syntax. Maybe you meant to use a different type of

[issue44313] Generate LOAD_ATTR+CALL_FUNCTION instead of LOAD_METHOD+CALL_METHOD for imports

2021-06-11 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- keywords: +patch pull_requests: +25263 stage: -> patch review pull_request: https://github.com/python/cpython/pull/26677 ___ Python tracker <https://bugs.python.org/issu

[issue44337] Port LOAD_ATTR to adaptive interpreter

2021-06-08 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: Mark would you mind sharing your opinions on this: https://bugs.python.org/issue44313 (should we still keep generating LOAD_METHOD/CALL_METHOD for potential objects imported through `from x import y`, or they also generate LOAD_ATTR/CALL_FUNCTION

[issue11105] Compiling recursive Python ASTs crash the interpreter

2021-06-08 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- stage: patch review -> resolved status: pending -> closed ___ Python tracker <https://bugs.python.org/issue11105> ___ ___

[issue11105] Compiling recursive Python ASTs crash the interpreter

2021-06-08 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: The issue has been solved, all buildbots should now pass. Will continue to monitor the situation. Thanks for the report @kj! -- priority: release blocker -> resolution: -> fixed status: open ->

[issue11105] Compiling recursive Python ASTs crash the interpreter

2021-06-08 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: New changeset 87f502231c6d5b04a4d8aa23fba24fcf5303aebb by Batuhan Taskaya in branch '3.9': [3.9] bpo-11105: reduce the recursion limit for tests. (GH-26605) https://github.com/python/cpython/commit/87f502231c6d5b04a4d8aa23fba24fcf5303aebb

[issue11105] Compiling recursive Python ASTs crash the interpreter

2021-06-08 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: New changeset bd6f0d3eadfe5623657db6aeb69b94d21f86f4a0 by Batuhan Taskaya in branch '3.10': [3.10] bpo-11105: reduce the recursion limit for tests. (GH-26607) https://github.com/python/cpython/commit/bd6f0d3eadfe5623657db6aeb69b94d21f86f4a0

[issue11105] Compiling recursive Python ASTs crash the interpreter

2021-06-08 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: New changeset 8004c4570b1d1277ea8754e22b5eb60e63f5026c by Batuhan Taskaya in branch 'main': bpo-11105: document the new test.support.infinite_recursion context manager (GH-26604) https://github.com/python/cpython/commit

[issue11105] Compiling recursive Python ASTs crash the interpreter

2021-06-08 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- pull_requests: +25190 pull_request: https://github.com/python/cpython/pull/26607 ___ Python tracker <https://bugs.python.org/issue11

[issue11105] Compiling recursive Python ASTs crash the interpreter

2021-06-08 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- pull_requests: +25188 pull_request: https://github.com/python/cpython/pull/26605 ___ Python tracker <https://bugs.python.org/issue11

[issue11105] Compiling recursive Python ASTs crash the interpreter

2021-06-08 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- pull_requests: +25187 pull_request: https://github.com/python/cpython/pull/26604 ___ Python tracker <https://bugs.python.org/issue11

[issue11105] Compiling recursive Python ASTs crash the interpreter

2021-06-08 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: New changeset e58d762c1fb4ad5e021d016c80c2bc4513632d2f by Batuhan Taskaya in branch 'main': bpo-11105: reduce the recursion limit for tests (GH-26550) https://github.com/python/cpython/commit/e58d762c1fb4ad5e021d016c80c2bc4513632d2f

[issue39573] [C API] Make PyObject an opaque structure in the limited C API

2021-06-06 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- nosy: +BTaskaya nosy_count: 10.0 -> 11.0 pull_requests: +25148 pull_request: https://github.com/python/cpython/pull/26550 ___ Python tracker <https://bugs.python.org/issu

[issue11105] Compiling recursive Python ASTs crash the interpreter

2021-06-05 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: After playing with it for a couple hours and without much success of creating a test environment (only using buildbots), I decided not to introduce hard limits. Even though they make the original tests to pass, they don't solve the problem overall and also

[issue11105] Compiling recursive Python ASTs crash the interpreter

2021-06-05 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- pull_requests: +25141 pull_request: https://github.com/python/cpython/pull/26554 ___ Python tracker <https://bugs.python.org/issue11

[issue11105] Compiling recursive Python ASTs crash the interpreter

2021-06-05 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- pull_requests: +25137 stage: -> patch review pull_request: https://github.com/python/cpython/pull/26550 ___ Python tracker <https://bugs.python.org/issu

[issue11105] Compiling recursive Python ASTs crash the interpreter

2021-06-05 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: > The stack size for a window build is currently set to 2MB, which is usually > lesser than *nix 8MB. So I think an easy solution is to increase the stack > size for windows builds. > I'm guessing release builds aren't affected

[issue11105] Compiling recursive Python ASTs crash the interpreter

2021-06-05 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- nosy: +lukasz.langa priority: normal -> release blocker resolution: fixed -> stage: resolved -> status: closed -> open ___ Python tracker <https://bugs.python

[issue11105] Compiling recursive Python ASTs crash the interpreter

2021-06-05 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: > Batuhan, can you take a look? Yes. -- ___ Python tracker <https://bugs.python.org/issue11105> ___ ___ Python-bugs-lis

[issue44313] Generate LOAD_ATTR+CALL_FUNCTION instead of LOAD_METHOD+CALL_METHOD for imports

2021-06-04 Thread Batuhan Taskaya
Batuhan Taskaya added the comment: @mark.shannon what do you think about doing this both for `import ` and `from import `. It will definitely simplify the implementation (no need to extra visitors or flags, just using the default DEF_IMPORT one https://github.com/isidentical/cpython

[issue44313] Generate LOAD_ATTR+CALL_FUNCTION instead of LOAD_METHOD+CALL_METHOD for imports

2021-06-04 Thread Batuhan Taskaya
New submission from Batuhan Taskaya : import foo def func(): return foo.bar() The snippet above will generate the following code; 2 0 LOAD_GLOBAL 0 (foo) 2 LOAD_METHOD 1 (bar) 4 CALL_METHOD 0 6

[issue11105] Compiling recursive Python ASTs crash the interpreter

2021-06-03 Thread Batuhan Taskaya
Change by Batuhan Taskaya : -- pull_requests: +25116 pull_request: https://github.com/python/cpython/pull/26522 ___ Python tracker <https://bugs.python.org/issue11

  1   2   3   4   5   6   7   >