[PATCH] D40285: [MS] AARCH64 cleanup default WIN macros
Martell Malone via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Nov 21 00:46:47 PST 2017
martell added a comment.
In https://reviews.llvm.org/D40285#931241, @mstorsjo wrote:
> ... so in general I wouldn't mind doing a change like this, but I'd like to make it consistent on ARM, i386 and x86_64 at the same time in that case, instead of just changing aarch64.
Thanks for the quick feedback :)
The easy one is to get rid of WIN64 because gcc doesn't even do that for mingw.
What are your thoughts here on WIN32, I would prefer to move it.
I remember a long time ago a lot of projects moved to _WIN32 when they discovered that it was not supposed to be defined.
I updated the diff to reflect removing WIN64
Repository:
rL LLVM
https://reviews.llvm.org/D40285
More information about the cfe-commits
mailing list