mapbox / mapbox/node-cpp-skel

Make default release mode build a "profiling" build

Ouverte
#117 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Langage dominant
C++
Étoiles
74
Forks
9
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

For the ability to profile our code with perf (and to generate the most readable flamegraphs) it is important to:

  • Compile with -fno-omit-frame-pointer (unless the compiler already defaults to using this flag?)
  • Compile with some debug symbols (unless you don't need detailed stack traces of your node module)

Doing this would move node-cpp-skel to doing a "profiling" build by default: https://github.com/mapbox/cpp/blob/master/glossary.md#profiling-build.

We can accomplish this by applying something like https://github.com/mapnik/node-mapnik/pull/804 to node-cpp-skel.

refs https://twitter.com/brendangregg/status/472869470675795968

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Inspectez la configuration de build de release par défaut du dépôt, puis comparez l’approche demandée avec node-mapnik PR #804. Confirmez que le build par défaut utilise -fno-omit-frame-pointer et inclut les symboles de débogage, et vérifiez que le module Node obtenu reste compilable et adapté aux flamegraphs de perf.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
cpp, node.js
Domaine
build-system, performance
Type d'issue
Fonctionnalité
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
45/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.