[PATCH] D147982: Account for PATCHABLE instrs in Branch Relaxation

Daniel Hoekwater via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 11 15:08:24 PDT 2023


dhoekwater added a comment.

In D147982#4259151 <https://reviews.llvm.org/D147982#4259151>, @paquette wrote:

> Because we can see clearly which MIR opcodes should not be outlined, and it defends us against other optimizer changes in earlier passes. Like, if the MIR test starts failing, we know 100% that it's an outliner issue and not some other pass dropping the X-ray ops or something.

Got it, thanks! I'll write that up real quick.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147982



More information about the llvm-commits mailing list