This is an archive of the discontinued LLVM Phabricator instance.

[InstCombine] Remove calloc transformations
ClosedPublic

Authored by xbolva00 on May 22 2018, 12:41 PM.

Details

Summary

Previous patch does not care if a value is changed between calloc and strlen. This needs to be removed from InstCombine and maybe moved to DSE later after some rework.

Diff Detail

Repository
rL LLVM

Event Timeline

xbolva00 created this revision.May 22 2018, 12:41 PM
efriedma accepted this revision.May 22 2018, 12:56 PM

LGTM assuming this is a straight revert of the previous patch.

This revision is now accepted and ready to land.May 22 2018, 12:56 PM
This revision was automatically updated to reflect the committed changes.
llvm/trunk/lib/Transforms/Utils/SimplifyLibCalls.cpp