[PATCH] D63439: Wordsmith the semantics of invariant.load

Sanjoy Das via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 17 09:25:07 PDT 2019


sanjoy created this revision.
sanjoy added reviewers: skatkov, ebrevnov.
Herald added subscribers: bixia, jlebar, mcrosier.
Herald added a project: LLVM.

Don't phrase the semantics in terms of the optimizer.  Instead have a
more straightforward execution based semantic.


Repository:
  rL LLVM

https://reviews.llvm.org/D63439

Files:
  docs/LangRef.rst


Index: docs/LangRef.rst
===================================================================
--- docs/LangRef.rst
+++ docs/LangRef.rst
@@ -8512,10 +8512,10 @@
 The optional ``!invariant.load`` metadata must reference a single
 metadata name ``<index>`` corresponding to a metadata node with no
 entries. If a load instruction tagged with the ``!invariant.load``
-metadata is executed, the optimizer may assume the memory location
-referenced by the load contains the same value at all points in the
-program where the memory location is known to be dereferenceable;
-otherwise, the behavior is undefined.
+metadata is executed, the memory location referenced by the load has
+to contain the same value at all points in the program where the
+memory location is dereferenceable; otherwise, the behavior is
+undefined.
 
 The optional ``!invariant.group`` metadata must reference a single metadata name
  ``<index>`` corresponding to a metadata node with no entries.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D63439.205099.patch
Type: text/x-patch
Size: 964 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190617/77b174c2/attachment.bin>


More information about the llvm-commits mailing list