diff options
| author | Qiongsi Wu <qwu@ibm.com> | 2023-07-11 13:10:08 -0400 |
|---|---|---|
| committer | Qiongsi Wu <qwu@ibm.com> | 2023-07-11 13:24:09 -0400 |
| commit | 0b66b3417c026e145708d0e20bfd05a72e79f12a (patch) | |
| tree | 07abf1188e3f0bc24e6ad97d5b152358422728bd /clang/lib/Lex/ModuleMap.cpp | |
| parent | e7c48ffde1c8137822a1b0a1ba6c4be5b4624aa6 (diff) | |
| download | llvm-0b66b3417c026e145708d0e20bfd05a72e79f12a.zip llvm-0b66b3417c026e145708d0e20bfd05a72e79f12a.tar.gz llvm-0b66b3417c026e145708d0e20bfd05a72e79f12a.tar.bz2 | |
[clang][AIX] Fix Overly Strict LTO Option Checking against `data-sections` when `mxcoff-roptr` is in Effect
The LTO `-mxcoff-roptr` [[ https://github.com/llvm/llvm-project/blob/c6b2d25927817bdeca99653ee3e66720f33ce3ae/clang/lib/Driver/ToolChains/CommonArgs.cpp#L750 | check ]] against data sections is overly strict and it ignores the fact that [[ https://github.com/llvm/llvm-project/blob/c6b2d25927817bdeca99653ee3e66720f33ce3ae/llvm/lib/LTO/LTOCodeGenerator.cpp#L427 | data sections is on by default on AIX ]], causing valid LTO compilation to fail when `-fdata-sections` is not explicitly specified.
This patch revises the check so that an error is reported only if data sections is explicitly turned off for LTO.
Reviewed By: hubert.reinterpretcast
Differential Revision: https://reviews.llvm.org/D152021
Diffstat (limited to 'clang/lib/Lex/ModuleMap.cpp')
0 files changed, 0 insertions, 0 deletions