Hi! On Wed, Jul 22, 2020 at 03:53:34PM +0200, Andrea Corallo wrote: > Andrew Pinski <pins...@gmail.com> writes: > > Can you give a simple example of what this patch does? > > Sure, this pass simply moves a sliding window over the insns trying to > make sure that we never have more then 'max_branch' branches for every > 'granule_size' insns. > > If too many branches are detected nops are added where considered less > armful to correct that.
Should that actually be a sliding window, or should there actually just not be more than N branches per aligned block of machine code? Like, per fetch group. Can you not use ASM_OUTPUT_ALIGN_WITH_NOP (or ASM_OUTPUT_MAX_SKIP_ALIGN even) then? GCC has infrastructure for that, already. Segher