Skip to content

Bugs with print_list pass #1410

Closed
Closed
@Smit-create

Description

@Smit-create

The pass is not that robust, see with the current main branch the following fails in the LLVM backend:

def test_print_list():
    a: list[str] = ["ab", "abc", "abcd"]

    print(a) # works
    print(a, a) # fails

test_print_list()

Also, that pass seems to be only working for single argument list type, not with nested tuple, see:

def test_print_list():
    a: list[tuple[i32, i32]] = [(1,2), (3, 4), (5, 6)]

    print(a) # fails

test_print_list()

That will require much work on the pass too.

Moreover, C, at present, doesn't use any passes and already has a design that will need slight modifications to make all this work.

I will fix those issues with the pass in a new PR soon to make it work for LLVM.

Originally posted by @Smit-create in #1405 (comment)

Metadata

Metadata

Assignees

Labels

asr_passASR pass related changes

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions