From 04ba344176b239c0abc0c9efc8b7524e551e58ed Mon Sep 17 00:00:00 2001 From: Eli Friedman Date: Wed, 16 Mar 2022 15:35:57 -0700 Subject: [PATCH] [CodeGen] Inline _byteswap_* builtins. As discussed in D57915. Fixes https://github.com/llvm/llvm-project/issues/39999 . Differential Revision: https://reviews.llvm.org/D121865 --- clang/lib/CodeGen/CGBuiltin.cpp | 5 ++++- clang/test/Headers/ms-arm64-intrin.cpp | 8 ++++---- 2 files changed, 8 insertions(+), 5 deletions(-) diff --git a/clang/lib/CodeGen/CGBuiltin.cpp b/clang/lib/CodeGen/CGBuiltin.cpp index eec4c8089c75..4ca0a1f3c03d 100644 --- a/clang/lib/CodeGen/CGBuiltin.cpp +++ b/clang/lib/CodeGen/CGBuiltin.cpp @@ -2932,7 +2932,10 @@ RValue CodeGenFunction::EmitBuiltinExpr(const GlobalDecl GD, unsigned BuiltinID, } case Builtin::BI__builtin_bswap16: case Builtin::BI__builtin_bswap32: - case Builtin::BI__builtin_bswap64: { + case Builtin::BI__builtin_bswap64: + case Builtin::BI_byteswap_ushort: + case Builtin::BI_byteswap_ulong: + case Builtin::BI_byteswap_uint64: { return RValue::get(emitUnaryBuiltin(*this, E, Intrinsic::bswap)); } case Builtin::BI__builtin_bitreverse8: diff --git a/clang/test/Headers/ms-arm64-intrin.cpp b/clang/test/Headers/ms-arm64-intrin.cpp index 76a424e0ce4c..3c37b8ad2f4f 100644 --- a/clang/test/Headers/ms-arm64-intrin.cpp +++ b/clang/test/Headers/ms-arm64-intrin.cpp @@ -2,7 +2,7 @@ // RUN: %clang_cc1 -triple arm64-windows -O1 \ // RUN: -fms-compatibility -fms-compatibility-version=17.00 \ -// RUN: -ffreestanding -fsyntax-only -Werror \ +// RUN: -fsyntax-only -Werror \ // RUN: -isystem %S/Inputs/include %s -S -o - -emit-llvm 2>&1 \ // RUN: | FileCheck %s @@ -14,16 +14,16 @@ void check_nop() { } unsigned short check_byteswap_ushort(unsigned short val) { -// CHECK: call i16 @_byteswap_ushort(i16 noundef %val) +// CHECK: call i16 @llvm.bswap.i16(i16 %val) return _byteswap_ushort(val); } unsigned long check_byteswap_ulong(unsigned long val) { -// CHECK: call i32 @_byteswap_ulong(i32 noundef %val) +// CHECK: call i32 @llvm.bswap.i32(i32 %val) return _byteswap_ulong(val); } unsigned __int64 check_byteswap_uint64(unsigned __int64 val) { -// CHECK: call i64 @_byteswap_uint64(i64 noundef %val) +// CHECK: call i64 @llvm.bswap.i64(i64 %val) return _byteswap_uint64(val); }