[PATCH] D21271: Fix `InstCombine` to not widen metadata on store-to-load forwarding

Amaury SECHET via llvm-commits llvm-commits at lists.llvm.org
Sun Jun 12 15:20:29 PDT 2016


deadalnix added a subscriber: deadalnix.
deadalnix added a comment.

Can you explain what the problem is ? This patch doesn't seems to do much more than moving the check for the value to be a load from within InstCombineLoadStoreAlloca to FindAvailableLoadedValue via an out parameter (which seems like an hidden dependency, I'm not super fan TBH).

What do I miss here ?


http://reviews.llvm.org/D21271





More information about the llvm-commits mailing list