[PATCH] D158392: [RISCV] Fix `vmsge{u}.vx` lowering by not adding the mask operand if `vd == v0`

Kiva Oyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 24 00:41:41 PDT 2023


imkiva added a comment.

In D158392#4605627 <https://reviews.llvm.org/D158392#4605627>, @craig.topper wrote:

> Does binutils have the same issue? https://godbolt.org/z/zMf6xqeo7

Probably yes.

But I still think the mask operand should be transitive as lowering a masked instruction to an unmasked one may break certain constraints (?) and look wired 😕. I will post further information here when I know more.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158392



More information about the llvm-commits mailing list