Skip to content

Commit

Permalink
[COFF, ARM64] Set the data type widths and the data layout string
Browse files Browse the repository at this point in the history
Summary: COFF ARM64 is LLP64 platform. So int is 4 bytes, long is 4 bytes and long long is 8 bytes.

Reviewers: compnerd, ruiu, rnk, efriedma

Reviewed By: compnerd, efriedma

Subscribers: efriedma, javed.absar, cfe-commits, aemerson, llvm-commits, kristof.beyls

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

llvm-svn: 308222
  • Loading branch information
Mandeep Singh Grang committed Jul 17, 2017
1 parent e9140e5 commit daa40b9
Show file tree
Hide file tree
Showing 2 changed files with 102 additions and 1 deletion.
15 changes: 14 additions & 1 deletion clang/lib/Basic/Targets.cpp
Expand Up @@ -6654,13 +6654,26 @@ class MicrosoftARM64TargetInfo
MicrosoftARM64TargetInfo(const llvm::Triple &Triple,
const TargetOptions &Opts)
: WindowsTargetInfo<AArch64leTargetInfo>(Triple, Opts), Triple(Triple) {

// This is an LLP64 platform.
// int:4, long:4, long long:8, long double:8.
WCharType = UnsignedShort;
IntWidth = IntAlign = 32;
LongWidth = LongAlign = 32;
DoubleAlign = LongLongAlign = 64;
LongDoubleWidth = LongDoubleAlign = 64;
LongDoubleFormat = &llvm::APFloat::IEEEdouble();
IntMaxType = SignedLongLong;
Int64Type = SignedLongLong;
SizeType = UnsignedLongLong;
PtrDiffType = SignedLongLong;
IntPtrType = SignedLongLong;

TheCXXABI.set(TargetCXXABI::Microsoft);
}

void setDataLayout() override {
resetDataLayout("e-m:w-i64:64-i128:128-n32:64-S128");
resetDataLayout("e-m:w-p:64:64-i32:32-i64:64-i128:128-n32:64-S128");
}

void getVisualStudioDefines(const LangOptions &Opts,
Expand Down
88 changes: 88 additions & 0 deletions clang/test/CodeGen/coff-aarch64-type-sizes.c
@@ -0,0 +1,88 @@
// RUN: %clang_cc1 -triple aarch64-windows -emit-llvm -w -o - %s | FileCheck %s

// CHECK: target datalayout = "e-m:w-p:64:64-i32:32-i64:64-i128:128-n32:64-S128"
// CHECK: target triple = "aarch64--windows-msvc"

int check_short() {
return sizeof(short);
// CHECK: ret i32 2
}

int check_int() {
return sizeof(int);
// CHECK: ret i32 4
}

int check_long() {
return sizeof(long);
// CHECK: ret i32 4
}

int check_longlong() {
return sizeof(long long);
// CHECK: ret i32 8
}

int check_int128() {
return sizeof(__int128);
// CHECK: ret i32 16
}

int check_fp16() {
return sizeof(__fp16);
// CHECK: ret i32 2
}

int check_float() {
return sizeof(float);
// CHECK: ret i32 4
}

int check_double() {
return sizeof(double);
// CHECK: ret i32 8
}

int check_longdouble() {
return sizeof(long double);
// CHECK: ret i32 8
}

int check_floatComplex() {
return sizeof(float _Complex);
// CHECK: ret i32 8
}

int check_doubleComplex() {
return sizeof(double _Complex);
// CHECK: ret i32 16
}

int check_longdoubleComplex() {
return sizeof(long double _Complex);
// CHECK: ret i32 16
}

int check_bool() {
return sizeof(_Bool);
// CHECK: ret i32 1
}

int check_wchar() {
return sizeof(__WCHAR_TYPE__);
// CHECK: ret i32 2
}

int check_wchar_unsigned() {
return (__WCHAR_TYPE__)-1 > (__WCHAR_TYPE__)0;
// CHECK: ret i32 1
}

enum Small {
Item
};

int foo() {
return sizeof(enum Small);
// CHECK: ret i32 4
}

0 comments on commit daa40b9

Please sign in to comment.