This is an archive of the discontinued LLVM Phabricator instance.

[MemCpyOpt] precommit test for D155406 (NFC)
ClosedPublic

Authored by khei4 on Jul 16 2023, 11:21 PM.

Diff Detail

Event Timeline

khei4 created this revision.Jul 16 2023, 11:21 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 16 2023, 11:21 PM
khei4 requested review of this revision.Jul 16 2023, 11:21 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 16 2023, 11:21 PM
khei4 updated this revision to Diff 540872.Jul 16 2023, 11:31 PM

fix wrong diffs for TODO comments

khei4 retitled this revision from [MemCpyOpt] precommit test for D155406 (NFC) to (WIP)[MemCpyOpt] precommit test for D155406 (NFC).Jul 17 2023, 1:54 AM
khei4 updated this revision to Diff 542791.Jul 21 2023, 12:57 AM
khei4 retitled this revision from (WIP)[MemCpyOpt] precommit test for D155406 (NFC) to [MemCpyOpt] precommit test for D155406 (NFC).

add

  • loop positive/negative cases
  • pcwalton's test cases for multi-bb https://reviews.llvm.org/D140089
  • pcwalton's test cases which require Def like concepts to reduce
khei4 edited the summary of this revision. (Show Details)Jul 21 2023, 12:58 AM
khei4 updated this revision to Diff 551405.Aug 18 2023, 12:19 AM

add

  • a test for BB, Instruction mixed post-dominator finding
khei4 updated this revision to Diff 551903.Aug 20 2023, 10:42 PM

add phi node test

nikic added a comment.Aug 21 2023, 7:24 AM

Is there a negative test where %dest is written before the memcpy (with some control flow)? Maybe I missed it, but it seems like most of the tests here work on %src.

khei4 added a comment.Aug 21 2023, 3:01 PM

Is there a negative test where %dest is written before the memcpy (with some control flow)? Maybe I missed it, but it seems like most of the tests here work on %src.

Thank you for the good catch! I should have added the dest mod before copying for the branch case!

khei4 updated this revision to Diff 552149.Aug 21 2023, 3:01 PM

add a case for branched dest mod before copy

nikic accepted this revision.Aug 22 2023, 3:50 AM

LGTM

This revision is now accepted and ready to land.Aug 22 2023, 3:50 AM
This revision was automatically updated to reflect the committed changes.