This is an archive of the discontinued LLVM Phabricator instance.

[InstCombine][NFC] add tests for printf("%s", str) --> puts(str)/noop transformation.
ClosedPublic

Authored by yurai007 on Apr 23 2021, 2:39 AM.

Diff Detail

Event Timeline

yurai007 requested review of this revision.Apr 23 2021, 2:39 AM
yurai007 created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptApr 23 2021, 2:39 AM
xbolva00 accepted this revision.Apr 23 2021, 5:21 AM
This revision is now accepted and ready to land.Apr 23 2021, 5:21 AM
This revision was landed with ongoing or failed builds.Apr 23 2021, 5:27 AM
This revision was automatically updated to reflect the committed changes.
thakis added a subscriber: thakis.Apr 23 2021, 5:36 AM

Looks like this breaks tests: http://45.33.8.238/linux/44909/step_12.txt

Please take a look, and if it takes a while to fix please revert for now.

xbolva00 added a comment.EditedApr 23 2021, 5:44 AM

Looks like this breaks tests: http://45.33.8.238/linux/44909/step_12.txt

Please take a look, and if it takes a while to fix please revert for now.

Regenerated. SHould be fixed now. @yurai007 you have old version of "opt" on your system. For the next time: Fetch latest sources of LLVM & run ninja -j8 && ninja install. Then you can use update scripts.