[PATCH] D156600: [llvm][NFC] Make test less brittle

Nathan Sidwell via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jul 29 12:22:17 PDT 2023


urnathan created this revision.
urnathan added a reviewer: nikic.
Herald added a subscriber: StephenFan.
Herald added a project: All.
urnathan requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

This test checks attributes by looking at the attribute dump.  That's both brittle and won't detect (some kinds of) attribute changes on functions. As it happens the memcpy changes I'm working on cause this test to spuriously fail as the attribute ordering is perturbed.

Make the test more robust by checking those on the actual function declarations.  In order to make the checks easier to write, it's better to run FileCheck twice -- once on stderr trace and once on the processed output.

Nikita -- you seem to have added the attribute checks?


https://reviews.llvm.org/D156600

Files:
  llvm/test/Analysis/BasicAA/cs-cs.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D156600.545380.patch
Type: text/x-patch
Size: 53321 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230729/d3969170/attachment.bin>


More information about the llvm-commits mailing list