Re: [PATCH] D24286: Add MS __nop intrinsic to intrin.h

2016-09-07 Thread Reid Kleckner via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL280826: Add MS __nop intrinsic to intrin.h (authored by rnk). Changed prior to commit: https://reviews.llvm.org/D24286?vs=70495&id=70555#toc Repository: rL LLVM https://reviews.llvm.org/D24286 Files

Re: [PATCH] D24286: Add MS __nop intrinsic to intrin.h

2016-09-07 Thread Nico Weber via cfe-commits
thakis accepted this revision. thakis added a comment. This revision is now accepted and ready to land. Looks great, thanks! https://reviews.llvm.org/D24286 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailma

[PATCH] D24286: Add MS __nop intrinsic to intrin.h

2016-09-06 Thread Albert Gutowski via cfe-commits
agutowski created this revision. agutowski added reviewers: rnk, thakis. agutowski added a subscriber: cfe-commits. There was no definition for __nop function - added inline assembly. https://reviews.llvm.org/D24286 Files: lib/Headers/intrin.h test/Headers/ms-intrin.cpp Index: lib/Headers/i