[all-commits] [llvm/llvm-project] 0b5396: [LegalizeVectorOps] Use all ones mask when expandi...

Craig Topper via All-commits all-commits at lists.llvm.org
Thu Apr 27 08:26:36 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 0b5396b163b6ba014679a6af365747303b95cf97
      https://github.com/llvm/llvm-project/commit/0b5396b163b6ba014679a6af365747303b95cf97
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2023-04-27 (Thu, 27 Apr 2023)

  Changed paths:
    M llvm/lib/CodeGen/SelectionDAG/LegalizeVectorOps.cpp

  Log Message:
  -----------
  [LegalizeVectorOps] Use all ones mask when expanding i1 VP_SELECT.

We were previously using the condition as the mask. By the semantics
of VP operations, that means that anywhere the condition is false
returns poison and not the false operand.

Use an all ones mask instead.

No tests are affected because RISC-V drops the mask when lowering.

Reviewed By: fakepaper56

Differential Revision: https://reviews.llvm.org/D149310




More information about the All-commits mailing list