This is an archive of the discontinued LLVM Phabricator instance.

Get compiler-rt/builtins to build on MSVC
AbandonedPublic

Authored by angelsl on Oct 2 2015, 7:02 AM.

Details

Reviewers
compnerd
Summary

Some compilers which target MSVC (e.g. Rust) depend on compiler-rt builtins to handle LLVM intrinsics like llvm.powi.*.

Make compiler-rt builtins compilable on MSVC.

Diff Detail

Event Timeline

angelsl updated this revision to Diff 36361.Oct 2 2015, 7:02 AM
angelsl retitled this revision from to Get compiler-rt/builtins to build on MSVC.
angelsl updated this object.
angelsl added a subscriber: llvm-commits.

This really feels like a number of different changes put into a single one. Please split this up.

lib/builtins/enable_execute_stack.c
26

If wtypes.h is needed/used, just include it. Don't wrap it in the check for already included.

30

Similar.

lib/builtins/int_math.h
75

This would be easier to follow if they were grouped rather than a single huge MSVC vs not block.

lib/builtins/int_types.h
168

clang-format the change

_[A-Z] is language reserved, please change the naming on these.

lib/builtins/int_util.h
30

This really complicates the usage. Im wondering if it may be useful to write this as a wapping macro instead.

Okay, I'll make the changes in a bit. Thanks.

angelsl abandoned this revision.Oct 3 2015, 10:27 AM

Split up into multiple patches.