Skip to content
Snippets Groups Projects

Forkify fixes

Merged Xavier Routh requested to merge forkify-fixes into main
1 unresolved thread

Merge request reports

Pipeline #201386 passed

Pipeline passed for 13bc8938 on forkify-fixes

Approval is optional

Merged by rarbore2rarbore2 1 month ago (Feb 4, 2025 3:55pm UTC)

Merge details

  • Changes merged into main with 0d5cd1ab (commits were squashed).
  • Deleted the source branch.
  • Auto-merge enabled

Pipeline #201387 passed

Pipeline passed for 0d5cd1ab on main

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
528 536 // PHIs on the frontier of the uses by the candidate phi, i.e in uses_for_dependance need
529 537 // to have headers that postdominate the loop continue latch. The value of the PHI used needs to be defined
530 538 // by the time the reduce is triggered (at the end of the loop's internal control).
539 // If anything in the intersection is a phi (that isn't this own phi), then the reduction cycle depends on control.
540 // Which is not allowed.
541 if intersection.iter().any(|cycle_node| editor.node(cycle_node).is_phi() && *cycle_node != *phi) || editor.node(loop_continue_latch).is_phi() {
542 return LoopPHI::ControlDependant(*phi);
543 }
  • Not allowed is a strong way to put it - it's not allowed:

    1. By the time of GCM
    2. Even then, only for non-parallel-reduce phis

    GCM has been modified to automatically unforkify fork-joins with control dependent, non-parallel reductions. So in this case, I believe we can actually forkify.

    Edited by rarbore2
  • Please register or sign in to reply
  • rarbore2
  • rarbore2 added 1 commit

    added 1 commit

    Compare with previous version

  • rarbore2 added 3 commits

    added 3 commits

    Compare with previous version

  • rarbore2 added 1 commit

    added 1 commit

    Compare with previous version

  • rarbore2 added 3 commits

    added 3 commits

    Compare with previous version

  • rarbore2 added 1 commit

    added 1 commit

    • 13bc8938 - use editor modified() fork forkify

    Compare with previous version

  • rarbore2 enabled an automatic merge when all merge checks for 13bc8938 pass

    enabled an automatic merge when all merge checks for 13bc8938 pass

  • rarbore2 mentioned in commit 0d5cd1ab

    mentioned in commit 0d5cd1ab

  • merged

  • Please register or sign in to reply
    Loading