Skip to content

[Refactor] SkipAligns cull

Summary

Now that SkipInstr includes alignment fields, this merge request seeks to remove arbitrary checks for them in the peephole optimizers, thereby resulting in more efficient and streamlined compiler code. The end result is the total removal of the SkipAligns method.

Additionally, the CollapseZeroDistJump method, which used to expect GetNextInstruction to stop on alignment fields and occasionally set p to equal one, now always returns the label and no longer needs specific checks for a tai_align afterwards.

System

  • Processor architecture: All (although x86 and AArch64 specifically used SkipAligns)

What is the current bug behavior?

N/A

What is the behavior after applying this patch?

The compiler should be slightly more efficient.

Edited by J. Gareth "Kit" Moreton

Merge request reports