[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