This is an archive of the discontinued LLVM Phabricator instance.

Use own struct instead of _Complex types on MSVC which lacks C99
ClosedPublic

Authored by angelsl on Oct 3 2015, 10:41 AM.

Details

Reviewers
compnerd

Diff Detail

Event Timeline

angelsl updated this revision to Diff 36438.Oct 3 2015, 10:41 AM
angelsl retitled this revision from to Use own struct instead of _Complex types on MSVC which lacks C99.
angelsl updated this object.
angelsl added a reviewer: compnerd.
angelsl added subscribers: compnerd, llvm-commits.
compnerd added inline comments.Oct 4 2015, 11:38 AM
lib/builtins/int_types.h
144

Is there a reason we can't use FComplex, DComplex, LComplex as names instead? This isn't a public header, so we don't have to worry about conflicting with user definitions.

angelsl updated this revision to Diff 36480.Oct 4 2015, 11:45 PM

Corrections made.

angelsl marked an inline comment as done.Oct 4 2015, 11:46 PM
compnerd added inline comments.Oct 5 2015, 7:21 PM
lib/builtins/int_types.h
158

Since the fields are hidden behind a macro anyways, any reason to not use "real" and "imaginary"?

angelsl updated this revision to Diff 36580.Oct 5 2015, 7:23 PM
angelsl marked an inline comment as done.
compnerd accepted this revision.Oct 5 2015, 7:26 PM
compnerd edited edge metadata.
This revision is now accepted and ready to land.Oct 5 2015, 7:26 PM
compnerd closed this revision.Oct 6 2015, 8:01 PM

SVN r249514.