Skip to content

[hls-fuzzer] Implement statistics for II - #991

Open
zero9178 wants to merge 3 commits into
mainfrom
users/zero9179/ii-statistic
Open

[hls-fuzzer] Implement statistics for II#991
zero9178 wants to merge 3 commits into
mainfrom
users/zero9179/ii-statistic

Conversation

@zero9178

Copy link
Copy Markdown
Collaborator

Multiple optimizations that we want to test are dependent on the II of the program. For this reason we want to be able to generate programs with either a wide variety or a specific bias regarding the II.

To measure this property, this PR adds a new statistic that displays the II achieved during simulation. The measurements shown are 1) a histogram that for a given II shows how many times that II occurred, the number of loops in total (i.e. number of measurements for II) and the median II.

Note that at the moment, if a loop is nested within another loop, then each loop iteration will be counted towards the II.

Depends on #988

@zero9178
zero9178 requested a review from Jiahui17 June 30, 2026 11:23
@zero9178
zero9178 force-pushed the users/zero9179/ii-instrumentation branch from bca04ee to 5ba2821 Compare July 29, 2026 11:59
@zero9178
zero9178 marked this pull request as draft July 29, 2026 12:48
@zero9178
zero9178 changed the base branch from users/zero9179/ii-instrumentation to main August 7, 2026 18:22
Multiple optimizations that we want to test are dependent on the II of the program. For this reason we want to be able to generate programs with either a wide variety or a specific bias regarding the II.

To measure this property, this PR adds a new statistic that displays the II achieved during simulation. The measurements shown are 1) a histogram that for a given II shows how many times that II occurred, the number of loops in total (i.e. number of measurements for II) and the median II.

Note that at the moment, if a loop is nested within another loop, then each loop iteration will be counted towards the II.
@zero9178
zero9178 force-pushed the users/zero9179/ii-statistic branch from 9841a0d to f6e32c2 Compare August 7, 2026 18:49
@zero9178
zero9178 marked this pull request as ready for review August 7, 2026 18:49
size_t pos = ref.find("II_INSTRUMENT:");
if (pos == llvm::StringRef::npos)
continue;

@Jiahui17 Jiahui17 Aug 9, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you pls add an example of the JSON being parsed here?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants