[PATCH] D131954: [LoongArch] Support Load and Store with 14-bit signed immediate operands

[Gong LingQin via Phabricator via llvm-commits]

gonglingqin added a comment.

In D131954#3730927 <https://reviews.llvm.org/D131954#3730927>, @xen0n wrote:

> In general you should name the testcases after your intent. For example for the `getelementptr ..., 255` case it's evident you want to check `{ld,st}ptr` is not emitted for misaligned offsets, and for `getelementptr ..., 4096` case you seem to be checking `{ld,st}ptr` is not emitted for out-of-range offsets. So names like `ldptr_w`, `ldptr_w_misaligned_offset` or `ldptr_w_too_big_offset` could be better conveying the intent.
>
> Otherwise LGTM.

Thanks, I will change it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131954