[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