[llvm] [SimplifyCFG][swifterror] Don't sink calls with swifterror params (PR #139015)

Nikita Popov via llvm-commits llvm-commits at lists.llvm.org
Thu May 8 01:00:01 PDT 2025


https://github.com/nikic commented:

I think it would be better to check this in canReplaceOperandWithVariable (and move the existing checks there as well).

In particular, legality here depends on whether sinking requires the introduction of a phi node or not. Could you add test variant where the swifterror argument is the same in both calls, and another argument differs?

https://github.com/llvm/llvm-project/pull/139015


More information about the llvm-commits mailing list