[clang] 04ba344 - [CodeGen] Inline _byteswap_* builtins.

[Eli Friedman via cfe-commits]

Author: Eli Friedman
Date: 2022-03-16T16:18:51-07:00
New Revision: 04ba344176b239c0abc0c9efc8b7524e551e58ed

URL: https://github.com/llvm/llvm-project/commit/04ba344176b239c0abc0c9efc8b7524e551e58ed
DIFF: https://github.com/llvm/llvm-project/commit/04ba344176b239c0abc0c9efc8b7524e551e58ed.diff

LOG: [CodeGen] Inline _byteswap_* builtins.

As discussed in D57915.

Fixes https://github.com/llvm/llvm-project/issues/39999 .

Differential Revision: https://reviews.llvm.org/D121865

Added: 
    

Modified: 
    clang/lib/CodeGen/CGBuiltin.cpp
    clang/test/Headers/ms-arm64-intrin.cpp

Removed: 
    


################################################################################
diff  --git a/clang/lib/CodeGen/CGBuiltin.cpp b/clang/lib/CodeGen/CGBuiltin.cpp
index eec4c8089c757..4ca0a1f3c03de 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 76a424e0ce4cd..3c37b8ad2f4f9 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);
 }