[PATCH] D159530: [RISCV] Add tests for memory constraint A

Alex Bradbury via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 19 04:19:46 PDT 2023


asb accepted this revision.
asb added a comment.
This revision is now accepted and ready to land.

LGTM, one minor nit. You're consistent with other tests in this file with using `nonnull` and `inbounds`, but I'd typically drop any attributes that don't alter the output of the test.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D159530



More information about the llvm-commits mailing list