fix clang build error on AARCH64 · Pull Request !530 · src-openEuler/lxc

Pull Requests

/

详情

已关闭

云沧:24.03-6 src-openEuler:openEuler-24.03-LTS

创建于 2024-09-15 01:31

克隆/下载

# PR主要描述信息

* 此PR基于[这个oEEP](https://gitee.com/openeuler/TC/blob/master/oEEP/oEEP-0003%20LLVM%E5%B9%B3%E8%A1%8C%E5%AE%87%E5%AE%99%E8%AE%A1%E5%88%92--%E5%9F%BA%E4%BA%8ELLVM%E6%8A%80%E6%9C%AF%E6%A0%88%E6%9E%84%E5%BB%BAoE%E8%BD%AF%E4%BB%B6%E5%8C%85.md)，旨在使得该软件包可以同时支持GCC和LLVM构建。
* 不影响原来的构建行为
* **问题一：**

```c
  [  112s] clang: warning: argument unused during compilation: '-fstack-clash-protection' [-Wunused-command-line-argument]
  [  112s] In file included from ../src/lxc/error.c:10:
  [  112s] ../src/lxc/log.h:310:6: error: conflicting types for 'strerror_r'
  [  112s]   310 |         int strerror_r(int errnum, char *buf, size_t buflen);
  [  112s]       |             ^
  [  112s] /usr/include/string.h:444:14: note: previous declaration is here
  [  112s]   444 | extern char *strerror_r (int __errnum, char *__buf, size_t __buflen)
  [  112s]       |              ^
  [  112s] 1 error generated.
  ```

* **问题成因总结：**  meson.build中有一处使用cc.compiles函数，本意是测试sterror_r的返回值类型（该函数在不同的系统中可能有不同的返回值类型），然而由于'-fstack-clash-protection'在aarch中暂时并未支持，再加上默认开启了‘-Werror’，因此导致编译结果为false。
* **修改建议：**  本意只是测试sterror_r的返回值类型，不应该让编译器选项产生干扰，因此额外给cc.compiles传入参数-Wno-error=unused-command-line-argument即可避免错误。

# 修复完毕

llvm构建工程：[Show home:yuncang:branches:Mega-LLVM:24.03 / lxc - 开源软件构建与测试 (isrc.ac.cn)](https://build.tarsier-infra.isrc.ac.cn/package/show/home:yuncang:branches:Mega-LLVM:24.03/lxc)

# 若仍有疑问，以下为问题分析与解决过程

### 问题一

已同步提交上游PR：[fix possible clang compile error on AARCH by yuncang123 · Pull Request #4479 · lxc/lxc (github.com)](https://github.com/lxc/lxc/pull/4479)

具体报错信息：

```c
[  112s] [2/1785] Compiling C object src/lxc/liblxc.a.p/error.c.o
[  112s] FAILED: src/lxc/liblxc.a.p/error.c.o 
[  112s] clang -Isrc/lxc/liblxc.a.p -Isrc/lxc -I../src/lxc -I. -I.. -Isrc -I../src -Isrc/include -I../src/include -I../src/lxc/json -I../src/lxc/cgroups -I../src/lxc/storage -fdiagnostics-color=always -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Wextra -std=gnu11 -O0 -g -include config.h -O2 -g -grecord-gcc-switches -pipe -fstack-protector-strong -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -Wp,-D_GLIBCXX_ASSERTIONS --config /usr/lib/rpm/generic-hardened-clang.cfg -fasynchronous-unwind-tables -fstack-clash-protection -fsigned-char -fPIC -pthread -fvisibility=default -MD -MQ src/lxc/liblxc.a.p/error.c.o -MF src/lxc/liblxc.a.p/error.c.o.d -o src/lxc/liblxc.a.p/error.c.o -c ../src/lxc/error.c
[  112s] clang: warning: argument unused during compilation: '-fstack-clash-protection' [-Wunused-command-line-argument]
[  112s] In file included from ../src/lxc/error.c:10:
[  112s] ../src/lxc/log.h:310:6: error: conflicting types for 'strerror_r'
[  112s]   310 |         int strerror_r(int errnum, char *buf, size_t buflen);
[  112s]       |             ^
[  112s] /usr/include/string.h:444:14: note: previous declaration is here
[  112s]   444 | extern char *strerror_r (int __errnum, char *__buf, size_t __buflen)
[  112s]       |              ^
[  112s] 1 error generated.
```

问题关键：

```c
[   94s] Checking if "strerror_r() returns char *" compiles: NO 
```

而这个结果本该是YES，因为openEuler-24.03-LTS中strerror_r的返回值类型就是char *没错。

问题出在：

```c
    code = '''
#define _GNU_SOURCE
#include <string.h>
int func (void) {
    char error_string[256];
    char *ptr = strerror_r (-2, error_string, 256);
    char c = *strerror_r (-2, error_string, 256);
    return c != 0 && ptr != (void*) 0L;
}
'''

have_func_strerror_r_char_p = cc.compiles(code, name : 'strerror_r() returns char *')
```

本意只是测试sterror_r的返回值类型，所以不应该让其他因素干扰cc.compiles的结果