This is an archive of the discontinued LLVM Phabricator instance.

utils/tdiff.py: Don't parse duplicate targets
ClosedPublic

Authored by luke on Jul 26 2023, 7:26 AM.

Details

Summary

When diffing a large target like all, most of the entries in the target
list in query_ninja are duplicated. I haven't looked that closely as to
why this is the case, but uniquing the list brought the size of it down
from 21781 to 76 for me.

Diff Detail

Repository
rT test-suite

Event Timeline

luke created this revision.Jul 26 2023, 7:26 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 26 2023, 7:27 AM
Herald added subscribers: asb, pmatos. · View Herald Transcript
luke requested review of this revision.Jul 26 2023, 7:27 AM
MatzeB accepted this revision.Jul 26 2023, 9:18 AM

LGTM

utils/tdiff.py
84
This revision is now accepted and ready to land.Jul 26 2023, 9:18 AM
luke marked an inline comment as done.Aug 4 2023, 3:17 AM

Thanks for writing this script btw, it's really handy :)

This revision was automatically updated to reflect the committed changes.