microsoft/GSL

Add performance tests to PR actions

Open

#1,167 opened on Nov 12, 2024

View on GitHub
 (10 comments) (0 reactions) (0 assignees)C++ (721 forks)batch import
Effort: MediumHelp WantedPriority: LowStatus: OpenType: Infra

Repository metrics

Stars
 (5,695 stars)
PR merge metrics
 (Avg merge 8d) (1 merged PR in 30d)

Description

It is important to maintain performance parity to std::span (at the very least when Safe Buffers are enabled). In order to track this, we should have performance tests between std::span and gsl::span for the latest standard library implementation across Clang, GCC, and MSVC.

@galenelias put together small benchmark comparing the two implementations on various workloads. They have been attached for convenience and provide a good starting point.

span_bench.zip

Contributor guide

Research direction
Examine the attached benchmark files and integrate them as performance tests in the CI pipeline to compare std::span and gsl::span across Clang, GCC, and MSVC.
Tech stack
cpp
Domain
testingperformance
Issue type
Test
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-2 days
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
C++performance testingCI/CD
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
40