[clang-tidy] NO.6 enable modernize-avoid-c-arrays check

[gouzil]

PR types

Others

PR changes

Others

Description

@GreatV 帮忙看看是否需要引入。

（这个也不能自动修复）

相关链接:

• 赛题四：在飞桨框架中引入 clang-tidy Tracking Issue #54073

[paddle-bot]

你的PR提交成功，感谢你对开源项目的贡献!
请关注后续CI自动化测试结果，详情请参考Paddle-CI手册。
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

[GreatV]

这个pytorch中是启用的，tensorflow中没有启用，跟pytorch对齐吧。

[gouzil]

这个pytorch中是启用的，tensorflow中没有启用，跟pytorch对齐吧。

我其实是担心会有一些性能上的损失

测试代码:

#include <iostream>
#include <array>

class demo1 {
    public:
        static constexpr uint32_t attributes_num = 1;
        static const char *attributes_name[attributes_num];
};

class demo2 {
    public:
        static constexpr uint32_t attributes_num = 1;
        const std::array<char*, attributes_num> attributes_name  = {"program"};
};

const char *demo1::attributes_name[demo1::attributes_num] = {"program"};

static void charbenchmark(benchmark::State& state) {
  for (auto _ : state) {
    demo1 d;
    benchmark::DoNotOptimize(d.attributes_name[0]);
  }
}
BENCHMARK(charbenchmark);

static void arraybenchmark(benchmark::State& state) {
  for (auto _ : state) {
    demo2 d2;
    benchmark::DoNotOptimize(d2.attributes_name[0]);
  }
}
BENCHMARK(arraybenchmark);

测试网站: https://quick-bench.com/

使用gcc8.1, -std=c++11

不开启 -O 优化

开启 -Os 优化

[GreatV]

release版一般会开启-O优化的吧

[zhangbo9674]

这里我们是必须要设定为 static，不建议修改

[gouzil]

存量过多拆分成几个pr来做，当前进度5%(可能还没到)

[SigureMo]

@gouzil 我看了下任务 7 cppcoreguidelines-avoid-c-arrays，文档里说它只是任务 6 modernize-avoid-c-arrays 的一个 alias，所以表单里任务 6、7 都记上你的名字了哈，最后完成时候同时启用这俩就行

@GreatV 另外，任务 1、2 和 8、9 好像也只是 alias 的关系，我在表单里做了一下标记以免认领冲突～

[luotao1]

需要解决下流水线的编译问题

[gouzil]

需要解决下流水线的编译问题

Done

[luotao1]

存量过多拆分成几个pr来做，当前进度5%(可能还没到)

当前commit还是进度5%不到么？

[SigureMo]

话说这些宏的话，是不是可以从宏定义处直接加 NOLINT 捏？不然可能要加很多 NOLINT

[gouzil]

有一部分是因为第三方库引起的

[gouzil]

这个我有在paddle/phi/core/flags.h的PHI_DEFINE_bool宏加上了，但是还是会扫到

[SigureMo]

这个我之前试的是，好像必须在相关语句的第一行才行，第二行好像就不太行

[gouzil]

确实，修复了

[gouzil]

存量过多拆分成几个pr来做，当前进度5%(可能还没到)

当前commit还是进度5%不到么？

是的，由于时间有点久了，先合入一部分然后我再扫一次，这个pr主要是验证可行性以及一些解决方案

[SigureMo]

LGTMeow