This is an archive of the discontinued LLVM Phabricator instance.

[CVP] Generate simpler code for elided with.overflow intrinsics
ClosedPublic

Authored by nikic on Aug 31 2019, 2:01 AM.

Details

Summary

Just a small cleanup... Use a { iN undef, i1 false } struct as the base, and only insert the first operand, instead of using { iN undef, i1 undef } as the base and inserting both. This is the same as what we do in InstCombine.

Diff Detail

Repository
rL LLVM

Event Timeline

nikic created this revision.Aug 31 2019, 2:01 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 31 2019, 2:01 AM
lebedev.ri accepted this revision.Aug 31 2019, 2:21 AM

This LG, thanks.

llvm/lib/Transforms/Scalar/CorrelatedValuePropagation.cpp
420 ↗(On Diff #218206)

// Rewrite this with.overflow intrinsic as non-overflowing ?

This revision is now accepted and ready to land.Aug 31 2019, 2:21 AM
This revision was automatically updated to reflect the committed changes.