llnl / llnl/RAJAPerf

Consider removing TSTEPS loops

Open
#538 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Jupyter Notebook
Stars
135
Forks
55
Avg merge
4d 16h
Merged PRs (30d)
5

Description

Do we need the TSTEPS loop in POLYBENCH_JACOBI_2D if we have the repeats loop? It would be easier to look at the performance of the parallel part of the kernel without the extra loop. I can see having the TSTEPS loop if we expect to incorporate it into the parallel loop.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the POLYBENCH_JACOBI_2D kernel and comparing its TSTEPS and repeats loops. Determine whether TSTEPS is still needed for the intended benchmark or should be incorporated into the parallel loop. Done means the loop structure and performance measurement behavior match the project's agreed intent.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
performance
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.