- Added more info about what we refer as a clobber in MSSA.
- Added more info about MemoryDefs and how there is a single Def chain.
- The doc portrayed MSSA as modeling the heap while it is modeling the whole memory, so I changed the wording to not be heap-specific.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
llvm/docs/MemorySSA.rst | ||
---|---|---|
25 | A lot of times, the combo "heap memory" is used instead of just "memory". It doesn't seem to me that MSSA is heap-specific, |
Thank you for the documentation improvements, much appreciated!
Looks good with a few nits inline.
llvm/docs/MemorySSA.rst | ||
---|---|---|
25 | It is not heap-specific. | |
101 | s/access/may access | |
109 | s/the use can't be hoisted outside/the use can't be hoisted outside alone. (It's possible they can be hoisted together; promotion to scalars does this.) |
Thanks for the comments!
- Addressed comments
- I also changed the wording from "heap" to "memory" in all places.
thanks for this!
if someone stamps with nits and you're confident that you've addressed the nits, feel free to commit :)
A lot of times, the combo "heap memory" is used instead of just "memory". It doesn't seem to me that MSSA is heap-specific,
do I miss something?