[PATCH] D146757: [Driver] Enable defining multilib flags verbatim

2023-05-25 Thread Michael Platings via Phabricator via cfe-commits
michaelplatings abandoned this revision. michaelplatings added a comment. Abandoning this in favour of D151437 & D151438 . Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D146757/new

[PATCH] D146757: [Driver] Enable defining multilib flags verbatim

2023-05-23 Thread Michael Platings via Phabricator via cfe-commits
michaelplatings added a comment. Funnily enough I had the same thought coming to work this morning that we don't need 2 syntaxes. I had an idea that we could only print flags beginning "-" but will consider the other options you suggested as well. Thanks. Repository: rG LLVM Github Monorepo

[PATCH] D146757: [Driver] Enable defining multilib flags verbatim

2023-05-23 Thread Petr Hosek via Phabricator via cfe-commits
phosek added a comment. I apologize about the delayed response. I had time to think about this a bit more and it's not really clear to me if we need to preserve the "plus/minus" flag syntax. Looking through the history, it seems that it was introduced in D2538 w

[PATCH] D146757: [Driver] Enable defining multilib flags verbatim

2023-05-03 Thread Michael Platings via Phabricator via cfe-commits
michaelplatings updated this revision to Diff 519005. michaelplatings added a comment. Re-run arc diff with clang-format in PATH Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D146757/new/ https://reviews.llvm.org/D146757 Files: clang/include/clan