Skip to content

[X86] Add support for MS inp functions. #93804

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
May 31, 2024
Merged

[X86] Add support for MS inp functions. #93804

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
de79bf7
Add support for MS inp functions.
MalaySanghi May 30, 2024
46fd6b1
reduce checks
MalaySanghi May 30, 2024
1899e28
fix opcode
MalaySanghi May 30, 2024
8e424c5
fix modifier
MalaySanghi May 31, 2024
31a7622
remove inp and inpw
MalaySanghi May 31, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 22 additions & 0 deletions clang/lib/Headers/intrin.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -329,6 +329,28 @@ static __inline__ void __DEFAULT_FN_ATTRS __stosq(unsigned __int64 *__dst,
static __inline__ void __DEFAULT_FN_ATTRS __halt(void) {
__asm__ volatile("hlt");
}

static inline int _inp(unsigned short port) {
int ret;
__asm__ volatile("inb %b1, %b0" : "=a"(ret) : "Nd"(port));
return ret;
}

static inline unsigned short _inpw(unsigned short port) {
unsigned short ret;
__asm__ volatile("inb %w1, %b0" : "=a"(ret) : "Nd"(port));
return ret;
}

static inline unsigned long _inpd(unsigned short port) {
unsigned long ret;
__asm__ volatile("inb %k1, %b0" : "=a"(ret) : "Nd"(port));
return ret;
}

#define inp(port) _inp((port))
#define inpw(port) _inpw((port))

#endif

#if defined(__i386__) || defined(__x86_64__) || defined(__aarch64__)
Expand Down
76 changes: 76 additions & 0 deletions clang/test/CodeGen/X86/ms-x86-intrinsics.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,82 @@ unsigned __int64 test__emulu(unsigned int a, unsigned int b) {
// CHECK: [[RES:%[0-9]+]] = mul nuw i64 [[Y]], [[X]]
// CHECK: ret i64 [[RES]]


int test_inp(unsigned short port) {
return _inp(port);
}
// CHECK-I386-LABEL: define dso_local i32 @test_inp(
// CHECK-I386-SAME: i16 noundef zeroext [[PORT:%.*]])
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These can be put in one line now.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, I had tried that. A variable (PORT in this case) is not allowed in CHECK-LABEL

// CHECK-I386-NEXT: entry:
// CHECK-I386-NEXT: [[TMP0:%.*]] = tail call i32 asm sideeffect "inb ${1:b}, ${0:b}", "={ax},N{dx},~{dirflag},~{fpsr},~{flags}"(i16 [[PORT]])
// CHECK-I386-NEXT: ret i32 [[TMP0]]
//
// CHECK-X64-LABEL: define dso_local i32 @test_inp(
// CHECK-X64-SAME: i16 noundef [[PORT:%.*]])
// CHECK-X64-NEXT: entry:
// CHECK-X64-NEXT: [[TMP0:%.*]] = tail call i32 asm sideeffect "inb ${1:b}, ${0:b}", "={ax},N{dx},~{dirflag},~{fpsr},~{flags}"(i16 [[PORT]])
// CHECK-X64-NEXT: ret i32 [[TMP0]]

unsigned short test_inpw(unsigned short port) {
return _inpw(port);
}
// CHECK-I386-LABEL: define dso_local zeroext i16 @test_inpw(
// CHECK-I386-SAME: i16 noundef zeroext [[PORT:%.*]])
// CHECK-I386-NEXT: entry:
// CHECK-I386-NEXT: [[TMP0:%.*]] = tail call i16 asm sideeffect "inb ${1:w}, ${0:b}", "={ax},N{dx},~{dirflag},~{fpsr},~{flags}"(i16 [[PORT]])
// CHECK-I386-NEXT: ret i16 [[TMP0]]
//
// CHECK-X64-LABEL: define dso_local i16 @test_inpw(
// CHECK-X64-SAME: i16 noundef [[PORT:%.*]])
// CHECK-X64-NEXT: entry:
// CHECK-X64-NEXT: [[TMP0:%.*]] = tail call i16 asm sideeffect "inb ${1:w}, ${0:b}", "={ax},N{dx},~{dirflag},~{fpsr},~{flags}"(i16 [[PORT]])
// CHECK-X64-NEXT: ret i16 [[TMP0]]

unsigned long test_inpd(unsigned short port) {
return _inpd(port);
}
// CHECK-I386-LABEL: define dso_local i32 @test_inpd(
// CHECK-I386-SAME: i16 noundef zeroext [[PORT:%.*]])
// CHECK-I386-NEXT: entry:
// CHECK-I386-NEXT: [[TMP0:%.*]] = tail call i32 asm sideeffect "inb ${1:k}, ${0:b}", "={ax},N{dx},~{dirflag},~{fpsr},~{flags}"(i16 [[PORT]])
// CHECK-I386-NEXT: ret i32 [[TMP0]]
//
// CHECK-X64-LABEL: define dso_local i32 @test_inpd(
// CHECK-X64-SAME: i16 noundef [[PORT:%.*]])
// CHECK-X64-NEXT: entry:
// CHECK-X64-NEXT: [[TMP0:%.*]] = tail call i32 asm sideeffect "inb ${1:k}, ${0:b}", "={ax},N{dx},~{dirflag},~{fpsr},~{flags}"(i16 [[PORT]])
// CHECK-X64-NEXT: ret i32 [[TMP0]]

int test_inp2(unsigned short port) {
return inp(port);
}
// CHECK-I386-LABEL: define dso_local i32 @test_inp2(
// CHECK-I386-SAME: i16 noundef zeroext [[PORT:%.*]])
// CHECK-I386-NEXT: entry:
// CHECK-I386-NEXT: [[TMP0:%.*]] = tail call i32 asm sideeffect "inb ${1:b}, ${0:b}", "={ax},N{dx},~{dirflag},~{fpsr},~{flags}"(i16 [[PORT]])
// CHECK-I386-NEXT: ret i32 [[TMP0]]
//
// CHECK-X64-LABEL: define dso_local i32 @test_inp2(
// CHECK-X64-SAME: i16 noundef [[PORT:%.*]])
// CHECK-X64-NEXT: entry:
// CHECK-X64-NEXT: [[TMP0:%.*]] = tail call i32 asm sideeffect "inb ${1:b}, ${0:b}", "={ax},N{dx},~{dirflag},~{fpsr},~{flags}"(i16 [[PORT]])
// CHECK-X64-NEXT: ret i32 [[TMP0]]

unsigned short test_inpw2(unsigned short port) {
return inpw(port);
}
// CHECK-I386-LABEL: define dso_local zeroext i16 @test_inpw2(
// CHECK-I386-SAME: i16 noundef zeroext [[PORT:%.*]])
// CHECK-I386-NEXT: entry:
// CHECK-I386-NEXT: [[TMP0:%.*]] = tail call i16 asm sideeffect "inb ${1:w}, ${0:b}", "={ax},N{dx},~{dirflag},~{fpsr},~{flags}"(i16 [[PORT]])
// CHECK-I386-NEXT: ret i16 [[TMP0]]
//
// CHECK-X64-LABEL: define dso_local i16 @test_inpw2(
// CHECK-X64-SAME: i16 noundef [[PORT:%.*]])
// CHECK-X64-NEXT: entry:
// CHECK-X64-NEXT: [[TMP0:%.*]] = tail call i16 asm sideeffect "inb ${1:w}, ${0:b}", "={ax},N{dx},~{dirflag},~{fpsr},~{flags}"(i16 [[PORT]])
// CHECK-X64-NEXT: ret i16 [[TMP0]]

#if defined(__x86_64__)

char test__readgsbyte(unsigned long Offset) {
Expand Down
Loading