[clang] [llvm] [RISC-V] Add support for MIPS P8700 CPU (PR #117865)

@@ -0,0 +1,371 @@ +//===- RISCVLoadStoreOptimizer.cpp ===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-Identifier: Ap

[clang] [llvm] [RISC-V] Add support for MIPS P8700 CPU (PR #117865)

@@ -0,0 +1,371 @@ +//===- RISCVLoadStoreOptimizer.cpp ===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-Identifier: Ap

[clang] [llvm] [RISC-V] Add support for MIPS P8700 CPU (PR #117865)

@@ -0,0 +1,371 @@ +//===- RISCVLoadStoreOptimizer.cpp ===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-Identifier: Ap

Re: [PATCH] D21959: [X86] Add xgetbv xsetbv intrinsics

guyblank added a comment. removing the MSC_VER check will not be enough, the feature guards from the intrinsic and the builtin need to be removed to make it work. not sure if this is the right way to go, any thoughts on this? Repository: rL LLVM https://reviews.llvm.org/D21959 __

Re: [PATCH] D21959: [X86] Add xgetbv xsetbv intrinsics

guyblank added a comment. Still, __XSAVE__ should have been defined when compiling for a target that supports the feature. But anyway, the xsaveintrin.h is quite small so always including it shouldn't be an issue. Are you ok with me removing the #if just for this header file, or would you like

r279042 - test commit

Author: guyblank Date: Thu Aug 18 03:44:33 2016 New Revision: 279042 URL: http://llvm.org/viewvc/llvm-project?rev=279042&view=rev Log: test commit Modified: cfe/trunk/lib/Basic/Targets.cpp Modified: cfe/trunk/lib/Basic/Targets.cpp URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Basic

Re: [PATCH] D21959: [X86] Add xgetbv xsetbv intrinsics

guyblank added a comment. Sorry about that, forgot that i changed the ms_intrin test. about the failure, I think that xsaveintrin.h is not being included because it requires the xsave feature - which should be on if the target supports it. do you know what in which target the failure occurred? a

Re: [PATCH] D21959: [X86] Add xgetbv xsetbv intrinsics

guyblank added a comment. If there aren't any further objections, I'd like go ahead with the commit. https://reviews.llvm.org/D21959 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D21959: [X86] Add xgetbv xsetbv intrinsics

guyblank added a comment. ping https://reviews.llvm.org/D21959 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D21959: [X86] Add xgetbv xsetbv intrinsics

guyblank updated this revision to Diff 65676. guyblank marked an inline comment as done. https://reviews.llvm.org/D21959 Files: include/clang/Basic/BuiltinsX86.def lib/CodeGen/CGBuiltin.cpp lib/Headers/intrin.h lib/Headers/xsaveintrin.h test/CodeGen/builtins-x86.c test/CodeGen/x86_32-

Re: [PATCH] D21959: [X86] Add xgetbv xsetbv intrinsics

guyblank added a comment. the include is because i added calls to the intrinsics themselves in the test, no just the builtins. Comment at: lib/Headers/intrin.h:905 @@ -906,9 +904,3 @@ } -static __inline__ unsigned __int64 __cdecl __DEFAULT_FN_ATTRS -_xgetbv(unsigned int __xcr

[PATCH] D21959: [X86] Add xgetbv xsetbv intrinsics

guyblank created this revision. guyblank added reviewers: aaboud, delena, craig.topper, AsafBadouh, m_zuckerman, igorb. guyblank added a subscriber: cfe-commits. [X86] Add xgetbv xsetbv intrinsics http://reviews.llvm.org/D21959 Files: include/clang/Basic/BuiltinsX86.def lib/CodeGen/CGBuilt