[PATCH] D29064: [MemorySSA] Add invariant.group handling

Daniel Berlin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 23 19:55:37 PST 2017


dberlin added a comment.

A few things

1. I'm curious why this isn't just added to the use optimizer more directly, it looks like you are computing info about most-dominating accesses, etc, which the stack already has.
2. If this isn't done in the walker, it'll never work in the face of updates.


https://reviews.llvm.org/D29064





More information about the llvm-commits mailing list