This is an archive of the discontinued LLVM Phabricator instance.

Move memory coercion functions from GVN.cpp to VNCoercion.cpp so they can be shared between GVN and NewGVN.
ClosedPublic

Authored by dberlin on Feb 28 2017, 3:17 PM.

Details

Summary

These are the functions used to determine when values of loads can be
extracted from stores, etc, and to perform the necessary insertions to
do this. There are no changes to the functions themselves except
reformatting, and one case where memdep was informed of a removed load
(which was pushed into the caller).

Diff Detail

Repository
rL LLVM

Event Timeline

dberlin created this revision.Feb 28 2017, 3:17 PM
davide accepted this revision.Mar 5 2017, 3:35 PM

LGTM, with the inline comment addressed. Thanks!

lib/Transforms/Utils/VNCoercion.cpp
1–6 ↗(On Diff #90093)

This probably needs an header comment with the license etc..

This revision is now accepted and ready to land.Mar 5 2017, 3:35 PM
This revision was automatically updated to reflect the committed changes.