google/benchmark

ICC sysinfo fixes

Open

#754 aberto em 16 de jan. de 2019

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)C++ (1.539 forks)batch import
good first issuehelp wanted

Métricas do repositório

Stars
 (7.968 stars)
Métricas de merge de PR
 (Mesclagem média 4d 2h) (19 fundiu PRs em 30d)

Description

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

Guia do colaborador

Direção de pesquisa
Adicione uma conversão explícita de hz para double na linha 558 do sysinfo.cc e adicione wd177 às flags do compilador ICC.
Pilha de tecnologia
cpp
Domain
tooling
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
90