[PATCH] D36362: [X86] Enable isel to use the PAUSE instruction even when SSE2 is disabled. Clang part

2017-08-05 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL310191: [X86] Enable isel to use the PAUSE instruction even when SSE2 is disabled. (authored by ctopper). Changed prior to commit: https://reviews.llvm.org/D36362?vs=109896=109902#toc Repository: rL

[PATCH] D36362: [X86] Enable isel to use the PAUSE instruction even when SSE2 is disabled. Clang part

2017-08-05 Thread Simon Pilgrim via Phabricator via cfe-commits
RKSimon accepted this revision. RKSimon added a comment. This revision is now accepted and ready to land. LGTM https://reviews.llvm.org/D36362 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D36362: [X86] Enable isel to use the PAUSE instruction even when SSE2 is disabled. Clang part

2017-08-05 Thread Craig Topper via Phabricator via cfe-commits
craig.topper updated this revision to Diff 109896. craig.topper added a comment. Add test case https://reviews.llvm.org/D36362 Files: include/clang/Basic/BuiltinsX86.def test/CodeGen/pause.c Index: test/CodeGen/pause.c ===

[PATCH] D36362: [X86] Enable isel to use the PAUSE instruction even when SSE2 is disabled. Clang part

2017-08-05 Thread Simon Pilgrim via Phabricator via cfe-commits
RKSimon added a comment. Codegen test case on no-sse2 target? https://reviews.llvm.org/D36362 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D36362: [X86] Enable isel to use the PAUSE instruction even when SSE2 is disabled. Clang part

2017-08-05 Thread Craig Topper via Phabricator via cfe-commits
craig.topper created this revision. On older processors this instruction encoding is treated as a NOP. MSVC doesn't disable intrinsics based on features the way clang/gcc does. Because the PAUSE instruction encoding doesn't crash older processors, some software out there uses these intrinsics