[PATCH] D139565: [IR] add new callbrpad instruction

Nick Desaulniers via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 7 17:03:40 PST 2022


nickdesaulniers added a comment.

In D139565#3979289 <https://reviews.llvm.org/D139565#3979289>, @efriedma wrote:

> If we do decide to go with an IR instruction, making the callbrpad optional seems likely to cause confusion in IR passes, in terms of what transformations are actually legal.

So I think this is why @void suggested I create a new instruction, rather than use an intrinsic (I have another impl of this patch using an intrinsic); by marking that instruction `isEHPad` most transforms know already not to touch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139565



More information about the llvm-commits mailing list