[PATCH] D135284: [Driver] select alternative target containing . in executable name

2022-10-26 Thread Dan McGregor via Phabricator via cfe-commits
dankm added a comment. In D135284#3886471 , @MaskRay wrote: > In D135284#3886469 , @dankm wrote: > >> Thanks for the feedback. I've added a testcase for this, I'll push that >> change and update the summary. > >

[PATCH] D135284: [Driver] select alternative target containing . in executable name

2022-10-26 Thread Dan McGregor via Phabricator via cfe-commits
dankm updated this revision to Diff 470918. dankm added a comment. Added a simple testcase, and updated commit message. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D135284/new/ https://reviews.llvm.org/D135284 Files: clang/lib/Driver/ToolChain.

[PATCH] D135284: [Driver] select alternative target containing . in executable name

2022-10-26 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added a comment. In D135284#3886469 , @dankm wrote: > Thanks for the feedback. I've added a testcase for this, I'll push that > change and update the summary. Did you upload a new patch to the differential? I cannot see it. Repository: rG LL

[PATCH] D135284: [Driver] select alternative target containing . in executable name

2022-10-26 Thread Dan McGregor via Phabricator via cfe-commits
dankm added a comment. Thanks for the feedback. I've added a testcase for this, I'll push that change and update the summary. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D135284/new/ https://reviews.llvm.org/D135284 _

[PATCH] D135284: [Driver] select alternative target containing . in executable name

2022-10-21 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added a comment. > The gcc compatible driver has support for selecting an alternative target > based on the driver's executable name, for instance > x86_64-unknown-linux-gnu-clang will set the target to linux on x86_64. I think the somewhat canonical term is target override (not selecti