[PATCH] D51217: [RISCV] atomic_store_nn have a different layout to regular store

Alex Bradbury via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 24 10:43:24 PDT 2018


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

Really good catch - thanks for the fix!



================
Comment at: lib/Target/RISCV/RISCVInstrInfoA.td:106
+defm : AtomicStPat<atomic_store_8,  SB, GPR>;
+defm : AtomicStPat<atomic_store_16,  SH, GPR>;
+defm : AtomicStPat<atomic_store_32, SW, GPR>;
----------------
Could you fix the extra whitespace after `atomic_store_16,` while you're doing this? i.e. ensuring SB/SH/SW are aligned.


https://reviews.llvm.org/D51217





More information about the llvm-commits mailing list