[PATCH] D120455: [CommandLine] Remove `may only occur zero or one times!` error

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 24 10:15:48 PST 2022


rnk added a comment.

I'm not familiar with gcc `--param`, but if this is it <https://chromium.googlesource.com/chromiumos/third_party/gcc/+/refs/heads/firmware-bolt_kirby-4979.B/gcc/params.def>, then it doesn't seem so bad. Nobody is suggesting that this be a stable, supported interface where we make long term commitments for LLVM option stability.

In any case, `-mllvm` already exists, and cl::opt is used by other components (MLIR). This change by itself is reasonable, and it doesn't stop us from working to reduce the exposed support surface of clang in the future. Having a slightly bad user experience for mllvm options doesn't actually reduce the support surface area of clang. We may have to agree to disagree here. :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120455



More information about the llvm-commits mailing list