[PATCH] D93786: [OpenMP][Fix] Make the arch selector for x86_64 work

2021-01-07 Thread Johannes Doerfert via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGd970a285b856: [OpenMP][Fix] Make the arch selector for x86_64 work (authored by jdoerfert). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93786/new/ https:/

[PATCH] D93786: [OpenMP][Fix] Make the arch selector for x86_64 work

2020-12-24 Thread Johannes Doerfert via Phabricator via cfe-commits
jdoerfert added a comment. In D93786#2470796 , @tianshilei1992 wrote: > Is it because the triple is separated by `-`? x86 and 64 is separated by - in the architecture list so `getArchTypeForLLVMName` will not recognize "x86_64". Repository: rG LLVM

[PATCH] D93786: [OpenMP][Fix] Make the arch selector for x86_64 work

2020-12-23 Thread Shilei Tian via Phabricator via cfe-commits
tianshilei1992 accepted this revision. tianshilei1992 added a comment. This revision is now accepted and ready to land. Is it because the triple is separated by `-`? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93786/new/ https://reviews.llvm.org/

[PATCH] D93786: [OpenMP][Fix] Make the arch selector for x86_64 work

2020-12-23 Thread Johannes Doerfert via Phabricator via cfe-commits
jdoerfert created this revision. jdoerfert added reviewers: tianshilei1992, jhuber6, JonChesterfield. Herald added subscribers: pengfei, guansong, bollu, hiraditya, yaxunl. jdoerfert requested review of this revision. Herald added subscribers: llvm-commits, cfe-commits, sstefan1. Herald added proje