[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