[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