Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • M mu-impl-fast
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 40
    • Issues 40
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 1
    • Merge requests 1
  • Deployments
    • Deployments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • mumu
  • mu-impl-fast
  • Merge requests
  • !27

Issue82 inf loop in peephole

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Yi Lin requested to merge issue82-inf-loop-in-peephole into develop Aug 31, 2017
  • Overview 0
  • Commits 4
  • Pipelines 0
  • Changes 6

Our peephole optimisation will remove jump-to-jump cases.

However, for cases such as

L1:
  jmp L2
..
L2:
  jmp L3
..
L3:
  jmp L1

the peephole optimiser will get into an infinite loop as we cannot find the final destination to jump. We need to be able to detect this infinite loop so the compiler won't get into an infinite loop. We do not apply optimisations (it is a busy infinite loop that does nothing, which is meaningless anyway).

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: issue82-inf-loop-in-peephole