This is an archive of the discontinued LLVM Phabricator instance.

[MSSA] Doc: Clobbers, more info on Defs / Def chain
ClosedPublic

Authored by baziotis on May 15 2020, 5:41 AM.

Details

Summary
  • 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.

Diff Detail

Event Timeline

baziotis created this revision.May 15 2020, 5:41 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 15 2020, 5:41 AM
baziotis marked an inline comment as done.May 15 2020, 5:41 AM
baziotis added inline comments.
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,
do I miss something?

simoll added a subscriber: simoll.May 15 2020, 6:19 AM
asbirlea accepted this revision.May 15 2020, 11:56 AM
asbirlea marked an inline comment as done.

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.)

This revision is now accepted and ready to land.May 15 2020, 11:56 AM
baziotis updated this revision to Diff 264375.May 15 2020, 3:53 PM
baziotis edited the summary of this revision. (Show Details)

Thanks for the comments!

  • Addressed comments
  • I also changed the wording from "heap" to "memory" in all places.

Ping :) Should I commit?

thanks for this!

if someone stamps with nits and you're confident that you've addressed the nits, feel free to commit :)

thanks for this!

if someone stamps with nits and you're confident that you've addressed the nits, feel free to commit :)

Got it, thanks :) I'm committing.

This revision was automatically updated to reflect the committed changes.