[PATCH] D145067: [AArch64] Precommit some more LD1R splat tests for scalar int/fp loads

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 1 12:00:01 PST 2023


SjoerdMeijer added a comment.

In D145067#4162019 <https://reviews.llvm.org/D145067#4162019>, @dmgreen wrote:

> Additional tests sounds OK. We don't always match splats to a single cost like we should.

The LD1R is a high latency instruction, the SWOG says a latency of 8.

> I tend to add many tests to one function because the amount of boilerplate code is otherwise quite high, but either way sounds fine (and most other costmodel tests are smaller sets of instructions).




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

https://reviews.llvm.org/D145067



More information about the llvm-commits mailing list