[PATCH] D27114: Preserve nonnull metadata on Loads through SROA & mem2reg.

Luqman Aden via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 19 13:46:40 PST 2016


luqmana added a comment.

In https://reviews.llvm.org/D27114#626971, @efriedma wrote:

> > I am quite sure I was told that !nonnull causes UB rather than poison. Is there some real documentation for that?
>
> Err, no, you're probably right; I haven't really worked with it, and LangRef doesn't say anything, so "must" defaults to undefined behavior.
>
> You still can't propagate backwards without proving post-dominance.


That should hold in this case no? Lines 361-381 assert that LI is dominated by the Store and the Store trivially dominates ReplVal?


https://reviews.llvm.org/D27114





More information about the llvm-commits mailing list