[PATCH] D47475: [Local] Make DoesKMove required for combineMetadata.

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 19 14:26:32 PDT 2018


efriedma added a comment.

> Also it allows sinking a load and preserving the nonnull metadata, if there are loads in all predecessors and they all have nonnull.

I would prefer to handle that specific case in some different way; it's not the same sort of reasoning.  In the CSE case, it doesn't matter whether J has nonnull metadata.


https://reviews.llvm.org/D47475





More information about the llvm-commits mailing list