[PATCH] D151211: [RISCV] Add test for small vector copies

Luke Lau via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 23 10:04:10 PDT 2023


luke added a comment.

In D151211#4364835 <https://reviews.llvm.org/D151211#4364835>, @reames wrote:

> LGTM, with the following additions:
>
> 1. Add one test with volatile load
> 2. Add one test with volatile store
> 3. Add one test with atomic load or store

Just to double check, it should be ok to perform scalarization on volatile/atomic stores right? That's what D150717 <https://reviews.llvm.org/D150717> already does


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151211



More information about the llvm-commits mailing list