[PATCH] D121740: [docs] Add details to MemorySSA docs.

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 17 01:38:56 PDT 2022


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

LG



================
Comment at: llvm/docs/MemorySSA.rst:286
+The second operand is optimized access, if there was a previous call on the
+walker's `getClobberingMemoryAccess(MA)`. This API will cache information
+as part of `MA`.
----------------
Need double backticks here.


================
Comment at: llvm/docs/MemorySSA.rst:305
+      }
+  }
 
----------------
I'd personally omit this example, and only keep the next one, to emphasize importance of the transitive use walk for correctness.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121740



More information about the llvm-commits mailing list