[PATCH] D143763: [Clang] Add clangMinimumVersion to multilib.yaml

2023-02-22 Thread Michael Platings via Phabricator via cfe-commits
michaelplatings abandoned this revision. michaelplatings marked 3 inline comments as done. michaelplatings added a comment. Squashed into D142932 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D143763/new/

[PATCH] D143763: [Clang] Add clangMinimumVersion to multilib.yaml

2023-02-13 Thread Peter Smith via Phabricator via cfe-commits
peter.smith added a comment. No objections to MaskRay's suggestion to merge with an earlier patch. I've made some suggestions to make the error messages be a bit more specific about what is wrong. Comment at: clang/lib/Driver/Multilib.cpp:185 +if

[PATCH] D143763: [Clang] Add clangMinimumVersion to multilib.yaml

2023-02-10 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added a comment. I think this patch can be merged into a previous one that introduces `variants` and `flagMap` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D143763/new/ https://reviews.llvm.org/D143763

[PATCH] D143763: [Clang] Add clangMinimumVersion to multilib.yaml

2023-02-10 Thread Michael Platings via Phabricator via cfe-commits
michaelplatings created this revision. michaelplatings added a reviewer: peter.smith. Herald added a subscriber: dmgreen. Herald added a project: All. michaelplatings requested review of this revision. Herald added subscribers: cfe-commits, MaskRay. Herald added a project: clang. This avoids a