microsoft/GSL

Add performance tests to PR actions

Open

#1.167 geöffnet am 12. Nov. 2024

Auf GitHub ansehen
 (8 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.695 Stars) (721 Forks)batch import
Effort: MediumHelp WantedPriority: LowStatus: OpenType: Infra

Beschreibung

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

Tech Stack
cpp
Domain
performancetesting
Issue Type
feature
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 days
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
fresh
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
C++performance testing
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
50
Research-Richtung
Implement performance benchmarks comparing gsl::span to std::span, using the attached span bench.zip as a starting point. The benchmarks should measure various workloads across Clang, GCC, and MSVC. Ensure the tests are integrated into the CI pipeline so that performance regression can be tracked automatically.