[PATCH] D134668: [LTO][clang] Using Single Dash Consistently when Passing LTO Options

Thu Oct 6 14:20:07 PDT 2022

qiongsiwu1 added a comment.

In D134668#3841312 <https://reviews.llvm.org/D134668#3841312>, @MaskRay wrote:

> I think I want to revert this change. `--plugin-opt=` is entirely fine with lld and GNU gold/ld... And generally the double-dash long options are preferred.

Sure. What we are aiming at is using single or double dashes consistently, so that when the AIX plugin prefix is introduced, we don't deal with arbitrary one or two dashes. I can work on a patch to change all `-plugin-opt=` to `--plugin-opt=`. How does having two dashes consistently sound?

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D134668/new/

https://reviews.llvm.org/D134668