This is an archive of the discontinued LLVM Phabricator instance.

[llvm][SelectionDAGBuilder] change callbr.landingpad intrinsic to accept explicit param
AbandonedPublic

Authored by nickdesaulniers on Jan 30 2023, 3:39 PM.

Details

Summary

This simplifies code in SelectionDAGBuilder. Originally, this intrinsic
was designed to have an implicit operand, based on
https://gist.github.com/nickdesaulniers/6df8e1a15f23636e2bf8abec96d425c7.

Turns out, codegen is simpler to just have an explicit parameter. Make
what was the implicit corresponding CallBrInst explicit and simplify
codegen.

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptJan 30 2023, 3:39 PM
nickdesaulniers requested review of this revision.Jan 30 2023, 3:39 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 30 2023, 3:39 PM
efriedma accepted this revision.Feb 6 2023, 9:28 AM

LGTM

This revision is now accepted and ready to land.Feb 6 2023, 9:28 AM
  • rebase, format
void accepted this revision.Feb 6 2023, 10:21 AM

LGTM