[PATCH] D82951: [ARM] Enable outliner at -Oz for M-class

[Yvan Roux via Phabricator via llvm-commits]

yroux added a comment.

Hi Sam,

no worries, I thought the optim would have to bake a bit longer before being bundled into Oz, but I'm happy you propose to do it and think it will be better tested with that change.

So I'm OK with your proposal and the patch, but while working on the next Oulining mode I found a latent issue with the liveness information calcuted by the MachineOutliner, in a nutshell if-conversion can introduce a predicated BX_RET in the middle of a BB and the current implementation doesn't see that LR is not dead and needs to be saved/restored when trying to outline a  chunk before this BX_RET.  I'm testing a fix for that and will submit it as soon as possible, so maybe wait for it before committing it.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D82951/new/

https://reviews.llvm.org/D82951