This is an archive of the discontinued LLVM Phabricator instance.

Clean up instcombine stpcpy test
ClosedPublic

Authored by lanza on Aug 4 2021, 8:09 PM.

Details

Summary

Deduplicate some code and add an additional test to verify that the
sprintf->stpcpy optimization still works on android21 (which properly
supports it).

This follows up 58481663692b55.

Diff Detail

Event Timeline

lanza created this revision.Aug 4 2021, 8:09 PM
lanza requested review of this revision.Aug 4 2021, 8:09 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 4 2021, 8:09 PM
smeenai accepted this revision.Aug 5 2021, 9:35 AM

LGTM, thanks!

This revision is now accepted and ready to land.Aug 5 2021, 9:35 AM
srhines accepted this revision.Aug 5 2021, 12:18 PM
srhines added a subscriber: srhines.

LGTM from the Android side. Thanks for finding and fixing the original issue, as well as adding this stronger test. :)

This revision was automatically updated to reflect the committed changes.