google/benchmark

ICC sysinfo fixes

Open

#754 创建于 2019年1月16日

在 GitHub 查看
 (1 评论) (0 反应) (0 负责人)C++ (1,539 fork)batch import
good first issuehelp wanted

仓库指标

Star
 (7,968 star)
PR 合并指标
 (平均合并 4天 2小时) (30 天内合并 19 个 PR)

描述

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

贡献者指南

研究方向
在 sysinfo.cc 第 558 行为 hz 添加显式转换为 double，并在 ICC 编译器标志中添加 wd177。
技术栈
cpp
领域
tooling
议题类型
缺陷
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
1
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1 小时以内
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
较久未更新
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
C++Git
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
90