[PATCH] D136432: [AMDGPU] Combine BFI instructions.

[Thomas Symalla via Phabricator via llvm-commits]

tsymalla added a comment.

In D136432#3873908 <https://reviews.llvm.org/D136432#3873908>, @jsilvanus wrote:

> I see, thanks!
>
> So `Select_BFI` just implements special case handling of BFI matching with more complicated expressions that we can still work with (because selectors are constants), and the default case is handled elsewhere.
> Maybe we should rename `Select_BFI` to reflect that?

Maybe `Selected_NestedBFI` might be more accurate


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136432