clang-cc.cpp fails to compile under MSVC

[llvmbot]

Bugzilla Link	4183
Resolution	FIXED
Resolved on	Feb 22, 2010 12:55
Version	unspecified
OS	Windows XP
Attachments	getTerminalWidth for Windows
Reporter	LLVM Bugzilla Contributor
CC	@DougGregor

Extended Description

/tools/clang-cc/clang-cc.cpp fails to compile under MSVC. On MSVC <sys/types.h> is available (and HAVE_SYS_TYPES_H is defined) but not <sys/ioctl.h>. Attached is a rough patch that uses HAVE_SYS_IOCTL_H. It still needs a configure check to actually define this.

This also adds a call to the equivalent function on Windows but this requires #including <windows.h>. If that's not something you want then this patch gets a lot simpler: just add "&& HAVE_SYS_IOCTL_H" to both of the existing "#if HAVE_SYS_TYPES_H" lines.

[llvmbot]

The right thing is to expose this via an llvm::sys API. I will try to stub this out & integrate your patch into it.

[llvmbot]

*** Bug llvm/llvm-bugzilla-archive#4191 has been marked as a duplicate of this bug. ***

[DougGregor]

That's for the patch! This should be integrated in r71450, but I can't test it locally. Please re-open this PR if the problem persists.

[llvmbot]

mentioned in issue llvm/llvm-bugzilla-archive#4191