[PATCH] D144341: [Driver][FreeBSD] Correct driver behavior if a triple is provided without a version

2023-02-21 Thread Brad Smith via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG2a2c0fd96757: [Driver][FreeBSD] Correct driver behavior if a triple is provided without a… (authored by brad). Repository: rG LLVM Github Monorepo

[PATCH] D144341: [Driver][FreeBSD] Correct driver behavior if a triple is provided without a version

2023-02-21 Thread Dimitry Andric via Phabricator via cfe-commits
dim accepted this revision. dim added a comment. This revision is now accepted and ready to land. In D144341#4140652 , @arichardson wrote: > LGTM, but I'll let @dim and @emaste decide if FreeBSD 8 code is still needed. No, these versions are ancient now

[PATCH] D144341: [Driver][FreeBSD] Correct driver behavior if a triple is provided without a version

2023-02-21 Thread Brad Smith via Phabricator via cfe-commits
brad updated this revision to Diff 499055. brad edited the summary of this revision. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D144341/new/ https://reviews.llvm.org/D144341 Files: clang/lib/CodeGen/TargetInfo.cpp clang/lib/Driver/ToolChains/F

[PATCH] D144341: [Driver][FreeBSD] Correct driver behavior if a triple is provided without a version

2023-02-21 Thread Alexander Richardson via Phabricator via cfe-commits
arichardson added a comment. LGTM, but I'll let @dim and @emaste decide if FreeBSD 8 code is still needed. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D144341/new/ https://reviews.llvm.org/D144341 ___ c

[PATCH] D144341: [Driver][FreeBSD] Correct usage of --hash-style=both with triple without version

2023-02-20 Thread Brad Smith via Phabricator via cfe-commits
brad updated this revision to Diff 498980. Herald added a subscriber: fedor.sergeev. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D144341/new/ https://reviews.llvm.org/D144341 Files: clang/lib/Driver/ToolChains/FreeBSD.cpp clang/test/Driver/cons

[PATCH] D144341: [Driver][FreeBSD]

2023-02-18 Thread Brad Smith via Phabricator via cfe-commits
brad created this revision. brad added a reviewer: dim. brad added a project: clang. Herald added subscribers: krytarowski, arichardson, emaste. Herald added a project: All. brad requested review of this revision. Herald added a subscriber: MaskRay. While looking at other usage of getOSMajorVersio