[all-commits] [llvm/llvm-project] b8741c: [VPlan] Relax assertion retrieving a scalar from V...

[Florian Hahn via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b8741cc185e89a95c9161218084f7c8b5578cf96
      https://github.com/llvm/llvm-project/commit/b8741cc185e89a95c9161218084f7c8b5578cf96
  Author: Florian Hahn <flo at fhahn.com>
  Date:   2024-07-19 (Fri, 19 Jul 2024)

  Changed paths:
    M llvm/lib/Transforms/Vectorize/VPlan.cpp
    M llvm/test/Transforms/LoopVectorize/SystemZ/scalar-steps-with-users-demanding-all-lanes-and-first-lane-only.ll

  Log Message:
  -----------
  [VPlan] Relax assertion retrieving a scalar from VPTransformState::get.

The current assertion VPTransformState::get when retrieving a single
scalar only does not account for cases where a def has multiple users,
some demanding all scalar lanes, some demanding only a single scalar.

For an example, see the modified test case. Relax the assertion by also
allowing requesting scalar lanes only when the Def doesn't have only its
first lane used.

Fixes https://github.com/llvm/llvm-project/issues/88849.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications