win: Fix another intrinsic include.

win: Fix another intrinsic include.

_mm_getcsr is declared in intrin.h, see
https://msdn.microsoft.com/en-us/library/hh977022.aspx
This worked because transitive includes ended up pulling in winnt.h,
which also declares this function (and tells the compiler to use a
built-in).  The built-in mechanism doesn't work with clang-cl
(see http://llvm.org/PR19898 comment 10) and leads to a linker error.
Since the file tries to include the right header for _mm_getcsr, just
fix the include, which happens to make things link with clang-cl too.

This should have been part of https://codereview.chromium.org/1778653002/

BUG=https://crbug.com/592745

Committed: https://chromium.googlesource.com/native_client/src/native_client/+/06cacd6becc3f033d03fab97e6fa9ef99d9b41f2

[Nico, 2016-03-17 16:37:30]

thakis@chromium.org changed reviewers:
+ mcgrathr@chromium.org

[Roland McGrath, 2016-03-17 18:38:53]

lgtm

[Nico, 2016-03-17 18:42:37]

The CQ bit was checked by thakis@chromium.org

[commit-bot: I haz the power, 2016-03-17 18:42:40]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1808123003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1808123003/1

[commit-bot: I haz the power, 2016-03-17 20:54:21]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-17 20:54:22]

Try jobs failed on following builders:
  nacl-win8-64_newlib_dbg on tryserver.nacl (JOB_FAILED,
http://build.chromium.org/p/tryserver.nacl/builders/nacl-win8-64_newlib_dbg/b...)

[Nico, 2016-03-17 22:16:42]

The CQ bit was checked by thakis@chromium.org

[commit-bot: I haz the power, 2016-03-17 22:16:49]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1808123003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1808123003/1

[commit-bot: I haz the power, 2016-03-17 23:23:26]

Description was changed from

==========
win: Fix another intrinsic include.

_mm_getcsr is declared in intrin.h, see
https://msdn.microsoft.com/en-us/library/hh977022.aspx
This worked because transitive includes ended up pulling in winnt.h,
which also declares this function (and tells the compiler to use a
built-in).  The built-in mechanism doesn't work with clang-cl
(see http://llvm.org/PR19898 comment 10) and leads to a linker error.
Since the file tries to include the right header for _mm_getcsr, just
fix the include, which happens to make things link with clang-cl too.

This should have been part of https://codereview.chromium.org/1778653002/

BUG=https://crbug.com/592745
==========

to

==========
win: Fix another intrinsic include.

_mm_getcsr is declared in intrin.h, see
https://msdn.microsoft.com/en-us/library/hh977022.aspx
This worked because transitive includes ended up pulling in winnt.h,
which also declares this function (and tells the compiler to use a
built-in).  The built-in mechanism doesn't work with clang-cl
(see http://llvm.org/PR19898 comment 10) and leads to a linker error.
Since the file tries to include the right header for _mm_getcsr, just
fix the include, which happens to make things link with clang-cl too.

This should have been part of https://codereview.chromium.org/1778653002/

BUG=https://crbug.com/592745

Committed:
https://chromium.googlesource.com/native_client/src/native_client/+/06cacd6be...
==========

[commit-bot: I haz the power, 2016-03-17 23:23:27]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/native_client/src/native_client/+/06cacd6be...