This is an archive of the discontinued LLVM Phabricator instance.

[IR] Add a test case for stripDebugLocFromLoopID
Needs ReviewPublic

Authored by kazu on Sep 10 2022, 2:33 PM.

Details

Summary

This patch adds a test case for
5a68d6721d042d57d1b2fef5d1b2e40a1260c169.

The credit for the test case goes to David.

Fixes https://github.com/llvm/llvm-project/issues/57557

Diff Detail

Event Timeline

kazu created this revision.Sep 10 2022, 2:33 PM
Herald added a project: Restricted Project. · View Herald TranscriptSep 10 2022, 2:33 PM
kazu requested review of this revision.Sep 10 2022, 2:33 PM
aprantl added inline comments.Sep 12 2022, 3:01 PM
llvm/test/DebugInfo/no-dicompileunit.ll
4

Can you also add a positive test? E.g.: CHECK: define {{.*}}@main otherwise this test will pass even if opt is a symlink to /bin/true

kazu updated this revision to Diff 459608.Sep 12 2022, 6:05 PM

Add a positive test.

kazu marked an inline comment as done.Sep 12 2022, 6:06 PM

Please take a look. Thanks!

kazu added a comment.Sep 14 2022, 9:06 AM

I'd appreciate a review. Thanks!

kazu added a comment.Sep 18 2022, 10:22 AM

Please take a look. Thanks!