[PATCH] D69841: Target Ivy bridge on macOS Mojave and later

2019-11-12 Thread David Zarzycki via Phabricator via cfe-commits
davezarzycki abandoned this revision. davezarzycki added a comment. Okay. I'm going to abandon this patch on the assumption that Apple has an internal bug tracking this. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69841/new/

[PATCH] D69841: Target Ivy bridge on macOS Mojave and later

2019-11-12 Thread Duncan P. N. Exon Smith via Phabricator via cfe-commits
dexonsmith requested changes to this revision. dexonsmith added a comment. This revision now requires changes to proceed. Hi Dave, thanks for checking in on this, but unfortunately we're not ready for this yet. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D69841: Target Ivy bridge on macOS Mojave and later

2019-11-12 Thread Alex Lorenz via Phabricator via cfe-commits
arphaman added reviewers: Gerolf, dexonsmith. arphaman added a comment. @bob.wilson no longer works on llvm-project. I added Gerolf and Duncan. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69841/new/ https://reviews.llvm.org/D69841

[PATCH] D69841: Target Ivy bridge on macOS Mojave and later

2019-11-12 Thread David Zarzycki via Phabricator via cfe-commits
davezarzycki added a comment. @bob.wilson – Ping. While Ivy Bridge (or really, Sandy Bridge) has AVX1, which enables faster memcpy, memset, and even memcmp (for common scenarios). If there is something I'm missing? If you can't explain why, that's okay. :-) Repository: rG LLVM Github

[PATCH] D69841: Target Ivy bridge on macOS Mojave and later

2019-11-05 Thread David Zarzycki via Phabricator via cfe-commits
davezarzycki added a comment. Hi @bob.wilson – Am I missing something here? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69841/new/ https://reviews.llvm.org/D69841 ___ cfe-commits mailing list

[PATCH] D69841: Target Ivy bridge on macOS Mojave and later

2019-11-05 Thread David Zarzycki via Phabricator via cfe-commits
davezarzycki created this revision. davezarzycki added a reviewer: bob.wilson. davezarzycki added a project: clang. Ivy Bridge is required by macOS 10.14 (Mojave) and later. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D69841 Files: clang/lib/Driver/ToolChains/Arch/X86.cpp