[PATCH] This patch introduces MemorySSA, a virtual SSA form for memory.Details on what it looks like are in MemorySSA.h

Daniel Berlin dberlin at dberlin.org
Sun Mar 1 21:01:56 PST 2015


On Mon, Mar 2, 2015 at 3:07 PM, David Majnemer <david.majnemer at gmail.com>
wrote:

> ================
> Comment at: include/llvm/Analysis/AliasAnalysis.h:155-156
> @@ +154,4 @@
> +      return getLocation(I);
> +    else
> +      llvm_unreachable("unsupported memory instruction");
> +  }
> ----------------
> No need for else here.


Will fix

>
>



> ================
> Comment at: include/llvm/Transforms/Utils/MemorySSA.h:1-2
> @@ +1,3 @@
> +//===- MemorySSA.h - Build Memory SSA ----------------------------*- C++
> +//-*-===//
> +//
> ----------------
> This looks strange, can you make it look like the other files?
>
>
Yeah, no idea what happened here.


> ================
> Comment at: lib/Transforms/Utils/MemorySSA.cpp:1
> @@ +1,2 @@
> +
> +//===- MemorySSA.cpp - Memory SSA
> Builder----------------------------------===//
> ----------------
> No need for this whitespace.
>
> http://reviews.llvm.org/D7864
>
> EMAIL PREFERENCES
>   http://reviews.llvm.org/settings/panel/emailpreferences/
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150302/a9c87817/attachment.html>


More information about the llvm-commits mailing list