diff options
| author | Qiongsi Wu <qiongsiwu@Qiongsis-MacBook-Air.local> | 2022-09-27 14:22:40 -0400 |
|---|---|---|
| committer | Qiongsi Wu <qiongsiwu@Qiongsis-MacBook-Air.local> | 2022-09-27 14:50:41 -0400 |
| commit | 86cd3535206d6db611260aae6f2869202eae4bb4 (patch) | |
| tree | 4f15da6e21028eafd6df1b80d4f99af14531d8c8 /openmp | |
| parent | 4fbbfd2530aae154a55ac873b6542c3222c25c51 (diff) | |
| download | llvm-86cd3535206d6db611260aae6f2869202eae4bb4.zip llvm-86cd3535206d6db611260aae6f2869202eae4bb4.tar.gz llvm-86cd3535206d6db611260aae6f2869202eae4bb4.tar.bz2 | |
[LTO][clang] Using Single Dash Consistently when Passing LTO Options
The following three static functions in `clang/lib/Driver/ToolChains/CommonArgs.cpp`
```
static void renderRpassOptions(...)
static void renderRemarksOptions(...)
static void renderRemarksHotnessOptions(...)
```
use `--plugin-opt` for the plugin option prefix, while the function `tools::addLTOOptions` uses `-plugin-opt`. This patch makes sure that we only use `-plugin-opt` (single dash) everywhere. It is not clear to me that why we decided to use `--plugin-opt` in https://reviews.llvm.org/D85810. If using `--plugin-opt` is intended, I'd love to hear the reason and I will close this patch.
We intend to followup this patch with a few other patches that teach `clang` to pass plugin options to the AIX linker, which uses a different prefix (`-bplugin_opt:`).
Reviewed By: w2yehia
Differential Revision: https://reviews.llvm.org/D134668
Diffstat (limited to 'openmp')
0 files changed, 0 insertions, 0 deletions