Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[REFACTOR][IR] Introduce SeqStmt to replace ir::Block #4627

Merged
merged 4 commits into from
Jan 6, 2020
Merged

[REFACTOR][IR] Introduce SeqStmt to replace ir::Block #4627

merged 4 commits into from
Jan 6, 2020

Commits on Jan 6, 2020

  1. [REFACTOR][IR] Introduce SeqStmt to replace Block 

    ir::Block was used to represent a sequence of Stmts in the original low-level IR.
The nested ir::Block structure is not really friendly for recursive visits,
especially when the statements are unrolled.

This PR introduce a SeqStmt that directly stores a sequence of statements in an Array container.
The new SeqStmt will be used as a replacement of the original Block structure.
    @tqchen
    tqchen committed Jan 6, 2020
    Configuration menu
    Copy the full SHA
    92c7ff6 View commit details
    Browse the repository at this point in the history

  2. [REFACTOR] Migrate use of Block to SeqStmt.

    @tqchen
    tqchen committed Jan 6, 2020
    Configuration menu
    Copy the full SHA
    0b91346 View commit details
    Browse the repository at this point in the history

  3. [REFACTOR] Remove Block

    @tqchen
    tqchen committed Jan 6, 2020
    Configuration menu
    Copy the full SHA
    0f308b6 View commit details
    Browse the repository at this point in the history

  4. Add more comments per yizhi's comment

    @tqchen
    tqchen committed Jan 6, 2020
    Configuration menu
    Copy the full SHA
    6a228dc View commit details
    Browse the repository at this point in the history