[PATCH] D80567: GlobalISel: Add a clarification to G_STORE documentation

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 26 08:39:01 PDT 2020


arsenm created this revision.
arsenm added reviewers: dsanders, aditya_nandakumar, aemerson, paquette.
Herald added subscribers: rovka, wdng.
Herald added a project: LLVM.

Mirror the note on G_LOAD. We probably do need to add an explicit
G_TRUNCSTORE opcode for the vector case, although I do not have a use
for it.


https://reviews.llvm.org/D80567

Files:
  llvm/docs/GlobalISel/GenericOpcode.rst


Index: llvm/docs/GlobalISel/GenericOpcode.rst
===================================================================
--- llvm/docs/GlobalISel/GenericOpcode.rst
+++ llvm/docs/GlobalISel/GenericOpcode.rst
@@ -567,7 +567,11 @@
 G_STORE
 ^^^^^^^
 
-Generic store. Expects a MachineMemOperand in addition to explicit operands.
+Generic store. Expects a MachineMemOperand in addition to explicit
+operands. If the stored value size is greater than the memory size,
+the high bits are implicitly truncated. If this is a vector store, the
+high elements are discarded (i.e. this does not function as a per-lane
+vector, truncating store)
 
 G_INDEXED_STORE
 ^^^^^^^^^^^^^^^


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D80567.266228.patch
Type: text/x-patch
Size: 663 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200526/fa9e0230/attachment.bin>


More information about the llvm-commits mailing list