google/benchmark

ICC sysinfo fixes

Open

#754 aperta il 16 gen 2019

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)C++ (1539 fork)batch import
good first issuehelp wanted

Metriche repository

Star
 (7968 star)
Metriche merge PR
 (Merge medio 4g 2h) (19 PR mergiate in 30 g)

Descrizione

sysinfo fails to compile on OSX with ICC due to two issues:

error #2259 is caused by the return value from sysinfo.cc:558 being implicitly cast from unsigned long long to double.

error #177 is caused by methods at sysinfo.cc:112 ValueUnion::GetAsString and sysinfo:114 ValueUnion::GetAsInteger being unused.

To resolve this we should:

  1. add an explicit cast for hz to double
  2. add -wd177 to the ICC compiler flags

Guida contributor

Direzione di ricerca
Aggiungi un cast esplicito per hz a double nella riga 558 di sysinfo.cc e aggiungi wd177 ai flag del compilatore ICC.
Tech stack
cpp
Dominio
tooling
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
90