[PATCH] D30154: MemorySSA: Add support for renaming uses in the updater.

Daniel Berlin via llvm-commits llvm-commits at lists.llvm.org
Sat Feb 18 22:24:10 PST 2017


On Sat, Feb 18, 2017 at 10:22 PM, Davide Italiano via Phabricator <
reviews at reviews.llvm.org> wrote:

> davide added a comment.
>
> The code logic looks fine. Some minor comments inline.
>
>
>
> ================
> Comment at: include/llvm/Transforms/Utils/MemorySSAUpdater.h:89-93
> +
>  private:
>    // Move What before Where in the MemorySSA IR.
>    template <class WhereType>
> +  void moveTo(MemoryUseOrDef *What, BasicBlock *BB, WhereType Where);
> ----------------
> Unrelated formatting, I guess.
>
>
> ================
> Comment at: lib/Transforms/Utils/MemorySSA.cpp:1187-1189
> +      // We already visited this during our renaming, which can happen
> when
> +      // being asked to rename multipleblocks. Figure out the incoming
> val,
> +      // which is the last def
> ----------------
> Full stop at the ned of the comment (here and everywhere else)
>
>
> I feel like i need to get emacs to check for this ...


> ================
> Comment at: unittests/Transforms/Utils/MemorySSA.cpp:193
>    MSSA.verifyMemorySSA();
> +  MSSA.dump();
>  }
> ----------------
> IIRC `dump()` is supposed to be used only in a debugger and `print()`
> should be used instead, but I might be wrong.
>
>
This was just there so i could reply to piotr's email :)
Removed.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170218/7d2d8b8f/attachment.html>


More information about the llvm-commits mailing list