Richard Sandiford wrote: > But why wasn't the index 0 as expected for the insns outside of the block?
Well it seems it checks for index 0 and sets the model_index as the current maximum model_index count. This means the target_bb check isn't strictly required - I build all of SPECINT2017 using the options from PR84068, and all 577k instances of instruction from another bb would sort correctly. So it looks we can remove the target_bb check. Wilco
