[all-commits] [llvm/llvm-project] db9057: [Sched] Skip MemOp with unknown size when clusteri...

[Pengcheng Wang via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: db9057edca0fe14987fb892f52bc51441316892c
      https://github.com/llvm/llvm-project/commit/db9057edca0fe14987fb892f52bc51441316892c
  Author: Pengcheng Wang <wangpengcheng.pp at bytedance.com>
  Date:   2024-12-05 (Thu, 05 Dec 2024)

  Changed paths:
    M llvm/lib/CodeGen/MachineScheduler.cpp

  Log Message:
  -----------
  [Sched] Skip MemOp with unknown size when clustering (#118443)

In #83875, we changed the type of `Width` to `LocationSize`. To get
the clsuter bytes, we use `LocationSize::getValue()` to calculate
the value.

But when `Width` is an unknown size `LocationSize`, an assertion
"Getting value from an unknown LocationSize!" will be triggered.

This patch simply skips MemOp with unknown size to fix this issue
and keep the logic the same as before.

This issue was found when implementing software pipeliner for
RISC-V in #117546. The pipeliner may clone some memory operations
with `BeforeOrAfterPointer` size.



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