This is an archive of the discontinued LLVM Phabricator instance.

Replace "CHECK-NOT: #{{.*}}" with same-line positive checks. NFC.
ClosedPublic

Authored by rprichard on Aug 5 2021, 5:22 PM.

Details

Summary

The intent of the negative #{{.*}} checks is to verify that the line
declaring/defining a function has no attribute, but they could restrict
later function declarations instead.

The 2008-09-02-FunctionNotes.ll check had allowed @fn3 to have an
attribute, because there is only a single "define void @fn3()" in the
output.

Diff Detail

Event Timeline

rprichard requested review of this revision.Aug 5 2021, 5:22 PM
rprichard created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptAug 5 2021, 5:22 PM

Follow up from D107527 review.

rprichard added inline comments.Aug 5 2021, 5:28 PM
llvm/test/Transforms/InferFunctionAttrs/annotate.ll
372

Whenever CHECK is selected with FileCheck --check-prefixes=, exactly one of CHECK-KNOWN or CHECK-UNKNOWN is also selected, so replacing the CHECK: on ffsl and ffsll isn't reducing the test coverage.

MaskRay accepted this revision.Aug 5 2021, 7:29 PM

LGTM.

This revision is now accepted and ready to land.Aug 5 2021, 7:29 PM