[x86] Fixed bug in new TEST optimisation where a FLAGS check always returned "in use" (!98) · Merge requests · FPC / FPC / FPC Source

J. Gareth "Kit" Moreton requested to merge CuriousKit/optimisations:test-merge-fix into main Dec 15, 2021

This merge request fixes the order that some instructions are removed in the TEST optimisations !95 (merged) and !96 (merged) - one of the FLAGS checks would always logically return "in use" because a Jcc instruction was only removed AFTER the check. Now, the conditional branch is removed first and hence RegUsedAfterInstruction can evaluate the use of FLAGS after that jump.

[x86] Fixed bug in new TEST optimisation where a FLAGS check always returned "in use"

Merge request reports