Re: [PATCH] D11808: Driver: Fix include and lib dirs when not using gcc under mingw

2015-08-12 Thread Martell Malone via cfe-commits
martell updated this revision to Diff 32007. martell added a comment. Updated to address yaron's comments. Okay for me to merge now ? http://reviews.llvm.org/D11808 Files: lib/Driver/MinGWToolChain.cpp test/Driver/Inputs/mingw_clang_tree/mingw32/i686-w64-mingw32/include/.keep

Re: r245084 - WindowsX86: long double is x87DoubleExtended on mingw

2015-08-19 Thread Martell Malone via cfe-commits
but I'm not sure it got through. Martell requested this be merged to 3.7. What do you think? LGTM Thanks! r245456. On Fri, Aug 14, 2015 at 12:05 PM, Martell Malone via cfe-commits cfe-commits@lists.llvm.org wrote: Author: martell Date: Fri Aug 14 14:05:56 2015

Re: r245459 - According to i686 ABI, long double size on x86 is 12 bytes not 16 bytes.

2015-08-20 Thread Martell Malone via cfe-commits
I feel very silly now. After testing the testcase again on svn it still works. It appears the OP was looking for this patch to go onto the 3.6 branch and was applying my patch to that. I'll know in future to recheck the testcase afterwards myself in future. Apologies for the noise guys. Yaron I

Re: [PATCH] D12037: WindowsX86: long double is x87DoubleExtended on mingw-w64

2015-08-14 Thread Martell Malone via cfe-commits
martell added a comment. Landed in http://reviews.llvm.org/rL245084 The commit parser is bugged :/ http://reviews.llvm.org/D12037 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D14179: Remove some legacy mingw-w64 gcc struct info

2015-10-29 Thread Martell Malone via cfe-commits
martell created this revision. martell added a reviewer: rnk. martell added a subscriber: cfe-commits. GCC versions starting at 4.8+ conform to standard Windows 64-bit ABI. Not quite sure if we should keep this to support mingw-w64 and gcc 4.6 Thoughts ? Also it seems to fix:

[PATCH] D14180: enable -fms-extensions by default on the mingw-w64 target

2015-10-29 Thread Martell Malone via cfe-commits
martell created this revision. martell added a reviewer: rnk. martell added subscribers: cfe-commits, yaron.keren, compnerd. As of commit b8a164 mingw-w64 support clang with -fms-extensions. We can built the mingw-w64 crt with clang now also. As we are dropping support for mingw.org I think

Re: [PATCH] D14180: enable -fms-extensions by default on the mingw-w64 target

2015-11-03 Thread Martell Malone via cfe-commits
martell added a comment. > This is what I'm worried about. :) Can you elaborate on what would break? Essentially mingw-w64 has a few header defines for working around builtings such as __rtdsc and the interlocked functions and some other builtins As soon as one of these headers are included it

Re: [PATCH] D14180: enable -fms-extensions by default on the mingw-w64 target

2015-11-03 Thread Martell Malone via cfe-commits
martell added a comment. In http://reviews.llvm.org/D14180#280184, @rnk wrote: > You're mostly interested in the non-builtin parts of -fms-extensions, like > declspec and UUID, right? Yes that would be one of the main reasons, gcc is very lacking in this department :) > You can then change

Re: [PATCH] D14179: Remove some legacy mingw-w64 gcc struct info

2015-11-03 Thread Martell Malone via cfe-commits
martell added a comment. From what I can gather it made sense at the time of gcc 4.6 so I don't think anyone is at fault :) Repository: rL LLVM http://reviews.llvm.org/D14179 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

r251930 - Remove some legacy mingw-w64 gcc struct info

2015-11-03 Thread Martell Malone via cfe-commits
Author: martell Date: Tue Nov 3 09:57:45 2015 New Revision: 251930 URL: http://llvm.org/viewvc/llvm-project?rev=251930=rev Log: Remove some legacy mingw-w64 gcc struct info As of gcc 4.7 mingw-w64 no longer emits 128-bit structs as i128 Differential Revision: http://reviews.llvm.org/D14179

Re: [PATCH] D14179: Remove some legacy mingw-w64 gcc struct info

2015-11-03 Thread Martell Malone via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL251930: Remove some legacy mingw-w64 gcc struct info (authored by martell). Changed prior to commit: http://reviews.llvm.org/D14179?vs=38744=39067#toc Repository: rL LLVM

Re: [PATCH] D14180: enable -fms-extensions by default on the mingw-w64 target

2015-11-03 Thread Martell Malone via cfe-commits
martell added a comment. In http://reviews.llvm.org/D14180#279651, @rnk wrote: > Will Clang be able to compile the last few stable mingw-w64 releases with > this change? I'd like that to keep working. Clang has not been able to compile mingw-w64 crt for the last few releases. I have only

[PATCH] D12466: Fix empty -L Path on OSX hosts

2015-08-28 Thread Martell Malone via cfe-commits
martell created this revision. martell added reviewers: rnk, yaron.keren. martell added a subscriber: cfe-commits. martell set the repository for this revision to rL LLVM. Not quite sure if this is the proper way to fix this. This is currently my work around when working from an OSX host though.

r245459 - According to i686 ABI, long double size on x86 is 12 bytes not 16 bytes.

2015-09-19 Thread Martell Malone via cfe-commits
For reference It was raised again here also https://github.com/Alexpux/MINGW-packages/issues/722 "that is extended from X86TargetInfo Which sets LongDoubleFormat = ::APFloat::x87DoubleExtended;" On Saturday, September

Re: [PATCH] D15006: Driver: Better detection of mingw-gcc

2015-12-09 Thread Martell Malone via cfe-commits
martell added a comment. hi ismail, I will tidy up for review closer towards the weekend where i can work on out of work stuff http://reviews.llvm.org/D15006 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D15006: Driver: Better detection of mingw-gcc

2015-12-15 Thread Martell Malone via cfe-commits
martell updated this revision to Diff 42933. martell added a comment. Sorry I was a bit slow getting around to this. Will do testcases if Yaron thinks this is okay http://reviews.llvm.org/D15006 Files: lib/Driver/MinGWToolChain.cpp lib/Driver/ToolChains.h Index: lib/Driver/ToolChains.h

r254117 - Driver: protect from empty -L args

2015-11-25 Thread Martell Malone via cfe-commits
Author: martell Date: Wed Nov 25 19:02:07 2015 New Revision: 254117 URL: http://llvm.org/viewvc/llvm-project?rev=254117=rev Log: Driver: protect from empty -L args Differential Revision: http://reviews.llvm.org/D12466 Modified: cfe/trunk/lib/Driver/ToolChain.cpp Modified:

Re: [PATCH] D12466: Fix empty -L Path on OSX hosts

2015-11-25 Thread Martell Malone via cfe-commits
martell added a comment. > Is this testable? Seems to me the only way to add this as a test would be to add bad code to clang to purposely pass an empty LibPath argument. This is probably something we do not want to do. This is probably why Reid never asked me for one in the review.

Re: [PATCH] D12466: Fix empty -L Path on OSX hosts

2015-11-25 Thread Martell Malone via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL254117: Driver: protect from empty -L args (authored by martell). Changed prior to commit: http://reviews.llvm.org/D12466?vs=33510=41202#toc Repository: rL LLVM http://reviews.llvm.org/D12466

Re: [PATCH] D15006: Driver: Better detection of mingw-gcc

2015-11-25 Thread Martell Malone via cfe-commits
martell added a comment. @ismail can you test this for your setup please? :) http://reviews.llvm.org/D15006 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D15006: Driver: Better detection of mingw-gcc

2015-11-25 Thread Martell Malone via cfe-commits
martell updated the summary for this revision. martell updated this revision to Diff 41208. martell added a comment. Removed the break that I didn't think was needed Can be re-added ?? Will added testcases http://reviews.llvm.org/D15006 Files: lib/Driver/MinGWToolChain.cpp Index:

Re: [PATCH] D15006: Driver: Better detection of mingw-gcc

2015-11-25 Thread Martell Malone via cfe-commits
martell updated this revision to Diff 41209. martell added a comment. Updated to reflect feedback I'm not too sure what todo for testcases on this one because they already exist for the most part. I ran the test suite and they all pass. I would have to create a mingw32-gcc and

Re: [PATCH] D14180: enable -fms-extensions by default on the mingw-w64 target

2015-11-30 Thread Martell Malone via cfe-commits
martell added a comment. I tried testing __has_builtin(_InterlockedCompareExchangeAdd) as it seem to not exist even when -fms-extensions is passed Is this a bug or it is intended ? http://reviews.llvm.org/D14180 ___ cfe-commits mailing list

r253815 - Test: Update mingw-useld.c to reflect r253813

2015-11-21 Thread Martell Malone via cfe-commits
Author: martell Date: Sat Nov 21 23:45:03 2015 New Revision: 253815 URL: http://llvm.org/viewvc/llvm-project?rev=253815=rev Log: Test: Update mingw-useld.c to reflect r253813 Modified: cfe/trunk/test/Driver/mingw-useld.c Modified: cfe/trunk/test/Driver/mingw-useld.c URL:

r253813 - Driver: Specifically tell the linker the target for mingw-w64

2015-11-21 Thread Martell Malone via cfe-commits
Author: martell Date: Sat Nov 21 23:40:06 2015 New Revision: 253813 URL: http://llvm.org/viewvc/llvm-project?rev=253813=rev Log: Driver: Specifically tell the linker the target for mingw-w64 Cross compiling from linux and OSX results in Error: Exec format. This is because the linker is expecting

r253874 - Revert part of r253813

2015-11-23 Thread Martell Malone via cfe-commits
Author: martell Date: Mon Nov 23 10:04:55 2015 New Revision: 253874 URL: http://llvm.org/viewvc/llvm-project?rev=253874=rev Log: Revert part of r253813 The new lld gnu frontend does not support the -target option Modified: cfe/trunk/lib/Driver/Tools.cpp Modified:

Re: r253813 - Driver: Specifically tell the linker the target for mingw-w64

2015-11-23 Thread Martell Malone via cfe-commits
7:05 AM, Rafael Espíndola < rafael.espind...@gmail.com> wrote: > This changes the linker from old-gnu to gnu. Are you sure you want to > that? I don't think the new linker takes a -target option. In any > case, it needs a test. > > Cheers, > Rafael > > > On 22 Novemb

r253898 - Driver: fallback to the location of clang if no sysroot,

2015-11-23 Thread Martell Malone via cfe-commits
Author: martell Date: Mon Nov 23 12:59:48 2015 New Revision: 253898 URL: http://llvm.org/viewvc/llvm-project?rev=253898=rev Log: Driver: fallback to the location of clang if no sysroot, hard coding /usr makes little sense for mingw-w64. If we have portable toolchains having /usr breaks that. If

Re: r253815 - Test: Update mingw-useld.c to reflect r253813

2015-11-23 Thread Martell Malone via cfe-commits
he patches in this series) look odd. > > The new "gnu" is ELF only. What exactly are you trying to do? > > Cheers, > Rafael > > > On 22 November 2015 at 00:45, Martell Malone via cfe-commits > <cfe-commits@lists.llvm.org> wrote: > > Author: martell >

Re: r253874 - Revert part of r253813

2015-11-23 Thread Martell Malone via cfe-commits
Yes the test case was updated in r253815 to reflect the change of old-gnu to gnu On Mon, Nov 23, 2015 at 8:37 AM, Rafael Espíndola < rafael.espind...@gmail.com> wrote: > This still needs a testcase. > > On 23 November 2015 at 11:04, Martell Malone via cfe-commits > <cfe-c

Re: r253898 - Driver: fallback to the location of clang if no sysroot,

2015-11-24 Thread Martell Malone via cfe-commits
onmez <ism...@i10z.com> wrote: > Hi, > > On Mon, Nov 23, 2015 at 8:59 PM, Martell Malone via cfe-commits > <cfe-commits@lists.llvm.org> wrote: > > Author: martell > > Date: Mon Nov 23 12:59:48 2015 > > New Revision: 253898 > > > > URL: http://llvm.o

Re: r253898 - Driver: fallback to the location of clang if no sysroot,

2015-11-24 Thread Martell Malone via cfe-commits
> > Why not hardcode /usr for Linux hosts? Portable toolchain packages exist but can't on linux if you do this. but in general I'm sure most would agree that detection is much better then hard coding. This is why Yaron approved the patch Well this is a custom clang it can be anywhere. Official

Re: r253898 - Driver: fallback to the location of clang if no sysroot,

2015-11-24 Thread Martell Malone via cfe-commits
> > Searching for gcc on Linux is not a good idea, you'll find the system one > not the mingw one. Searching for i686-w64-mingw32-gcc or > x86_64-w64-mingw32-gcc should work better, that is searching for: > getTriple().getArchName()) + "-w64-mingw32-gcc" Moreover, this should work on Windows as

r300562 - mingw-w64: enable support for __declspec(selectany)

2017-04-18 Thread Martell Malone via cfe-commits
Author: martell Date: Tue Apr 18 10:56:24 2017 New Revision: 300562 URL: http://llvm.org/viewvc/llvm-project?rev=300562=rev Log: mingw-w64: enable support for __declspec(selectany) Add selectany as a GCC spelling for mingw-w64 Reviewers: rnk Differential revision:

Re: r300555 - Driver: Better detection of mingw-gcc

2017-04-18 Thread Martell Malone via cfe-commits
wrote: > I think you've got the wrong Phabricator link in the commit log. > > On 18 April 2017 at 15:27, Martell Malone via cfe-commits < > cfe-commits@lists.llvm.org> wrote: > >> Author: martell >> Date: Tue Apr 18 09:27:36 2017 >> New Revision: 300555 >&

r300555 - Driver: Better detection of mingw-gcc

2017-04-18 Thread Martell Malone via cfe-commits
Author: martell Date: Tue Apr 18 09:27:36 2017 New Revision: 300555 URL: http://llvm.org/viewvc/llvm-project?rev=300555=rev Log: Driver: Better detection of mingw-gcc Stop blindly searching for "gcc.exe" on windows. Stop assuming "/usr" on unix, fixes cross compiling. Reviewers: mati865,

[libcxxabi] r302824 - [Libcxxabi]: Support using compiler-rt for MinGW64

2017-05-11 Thread Martell Malone via cfe-commits
Author: martell Date: Thu May 11 16:16:29 2017 New Revision: 302824 URL: http://llvm.org/viewvc/llvm-project?rev=302824=rev Log: [Libcxxabi]: Support using compiler-rt for MinGW64 Reviewers: EricWF Differential Revision: https://reviews.llvm.org/D33098 Modified:

Re: r304012 - Enable __float128 for mingw for GCC compatibility and define __SIZEOF_FLOAT128__ on x86

2017-05-26 Thread Martell Malone via cfe-commits
Thanks Reid for looking at this today On Fri, May 26, 2017 at 6:38 PM, Reid Kleckner via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: rnk > Date: Fri May 26 12:38:15 2017 > New Revision: 304012 > > URL: http://llvm.org/viewvc/llvm-project?rev=304012=rev > Log: > Enable __float128

[libcxxabi] r304026 - [libcxxabi] iconv is not needed for mingw-w64

2017-05-26 Thread Martell Malone via cfe-commits
Author: martell Date: Fri May 26 14:39:39 2017 New Revision: 304026 URL: http://llvm.org/viewvc/llvm-project?rev=304026=rev Log: [libcxxabi] iconv is not needed for mingw-w64 Modified: libcxxabi/trunk/cmake/config-ix.cmake Modified: libcxxabi/trunk/cmake/config-ix.cmake URL:

[libcxx] r303928 - libcxx: fix bootstrapping for mingw-w64

2017-05-25 Thread Martell Malone via cfe-commits
Author: martell Date: Thu May 25 17:37:15 2017 New Revision: 303928 URL: http://llvm.org/viewvc/llvm-project?rev=303928=rev Log: libcxx: fix bootstrapping for mingw-w64 Differential Revision: https://reviews.llvm.org/D33388 Modified: libcxx/trunk/cmake/config-ix.cmake

[libcxxabi] r304110 - [libcxxabi] Disable DLL annotations on static

2017-05-28 Thread Martell Malone via cfe-commits
Author: martell Date: Sun May 28 17:46:50 2017 New Revision: 304110 URL: http://llvm.org/viewvc/llvm-project?rev=304110=rev Log: [libcxxabi] Disable DLL annotations on static rL288692 renames _LIBCXXABI_DISABLE_DLL_IMPORT_EXPORT to _LIBCXXABI_DISABLE_VISIBILITY_ANNOTATIONS Modified:

[libcxxabi] r304374 - [libcxxabi] Rework CMakeLists.txt into modules

2017-05-31 Thread Martell Malone via cfe-commits
Author: martell Date: Wed May 31 19:09:20 2017 New Revision: 304374 URL: http://llvm.org/viewvc/llvm-project?rev=304374=rev Log: [libcxxabi] Rework CMakeLists.txt into modules Refactor cmake to remove dependence on LLVM's cmake modules. This improves handling of cmake checks when cross compiling

[libcxxabi] r304664 - [libcxxabi] HandleLLVMOptions in out of tree build

2017-06-03 Thread Martell Malone via cfe-commits
Author: martell Date: Sat Jun 3 12:23:19 2017 New Revision: 304664 URL: http://llvm.org/viewvc/llvm-project?rev=304664=rev Log: [libcxxabi] HandleLLVMOptions in out of tree build Differential revision: https://reviews.llvm.org/D33753 Modified:

r315864 - [driver] [cl] Add/fix c++17/c++latest

2017-10-15 Thread Martell Malone via cfe-commits
Author: martell Date: Sun Oct 15 10:27:58 2017 New Revision: 315864 URL: http://llvm.org/viewvc/llvm-project?rev=315864=rev Log: [driver] [cl] Add/fix c++17/c++latest Patch by: daxpedda Differential Revision: https://reviews.llvm.org/D38123 Modified:

r315867 - Driver: use ld64.lld when -fuse-ld=lld for darwin

2017-10-15 Thread Martell Malone via cfe-commits
Author: martell Date: Sun Oct 15 10:53:45 2017 New Revision: 315867 URL: http://llvm.org/viewvc/llvm-project?rev=315867=rev Log: Driver: use ld64.lld when -fuse-ld=lld for darwin When using lld on macOS the current level of detection between ld and ld64 forces us to rename lld to ld. For ELF

r315868 - Fix test case regresstion from rL315864

2017-10-15 Thread Martell Malone via cfe-commits
Author: martell Date: Sun Oct 15 11:01:28 2017 New Revision: 315868 URL: http://llvm.org/viewvc/llvm-project?rev=315868=rev Log: Fix test case regresstion from rL315864 The patch should have been checking against STDCXX17 not STDCXX14. Modified: cfe/trunk/test/Driver/cl-options.c Modified:

[libcxx] r313084 - libcxx: fix merge_archives error variable name

2017-09-12 Thread Martell Malone via cfe-commits
Author: martell Date: Tue Sep 12 15:32:02 2017 New Revision: 313084 URL: http://llvm.org/viewvc/llvm-project?rev=313084=rev Log: libcxx: fix merge_archives error variable name Differential Revision: https://reviews.llvm.org/D37728 Modified: libcxx/trunk/utils/merge_archives.py Modified:

Re: r313088 - Revert "[Driver] MinGW: Remove custom linker detection"

2017-09-12 Thread Martell Malone via cfe-commits
please explain why you are reverting it in the >>> commit message. >>> >>> On Tue, Sep 12, 2017 at 3:58 PM, Martell Malone via cfe-commits < >>> cfe-commits@lists.llvm.org> wrote: >>> >>>> Author: martell >>>> Date: Tue Sep 12 15:58:12 2

r313102 - [Driver] MinGW: Remove custom linker detection

2017-09-12 Thread Martell Malone via cfe-commits
Author: martell Date: Tue Sep 12 17:46:54 2017 New Revision: 313102 URL: http://llvm.org/viewvc/llvm-project?rev=313102=rev Log: [Driver] MinGW: Remove custom linker detection In rL289668 the ability to specify the default linker at compile time was added but because the MinGW driver used custom

r313104 - Revert "[Driver] MinGW: Remove custom linker detection"

2017-09-12 Thread Martell Malone via cfe-commits
Author: martell Date: Tue Sep 12 17:57:50 2017 New Revision: 313104 URL: http://llvm.org/viewvc/llvm-project?rev=313104=rev Log: Revert "[Driver] MinGW: Remove custom linker detection" This reverts rL313102 because it still fails some build bot tests. On many linux bots it fails with the

Re: r313082 - [Driver] MinGW: Remove custom linker detection

2017-09-12 Thread Martell Malone via cfe-commits
> Hi, >> Would you please have a look at: >> http://green.lab.llvm.org/green/job/clang-stage1-cmake-RA-in >> cremental/42380/ >> >> it appears this commit is causing some tests to fail on our bots. Any >> assistance you may provide in getting the bots back gre

Re: r313104 - Revert "[Driver] MinGW: Remove custom linker detection"

2017-09-12 Thread Martell Malone via cfe-commits
lld is remapped to lld-link is in lib/Driver/ToolChains/MSVC.cpp and that is only for msvc targets with a testcase in test/Driver/cl-link.c Best, Martell On Wed, Sep 13, 2017 at 1:57 AM, Martell Malone via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: martell > Date: T

Re: r313104 - Revert "[Driver] MinGW: Remove custom linker detection"

2017-09-12 Thread Martell Malone via cfe-commits
lld is remapped to lld-link is in lib/Driver/ToolChains/MSVC.cpp > and that is only for msvc targets with a testcase in test/Driver/cl-link.c > > Best, > Martell > > > On Wed, Sep 13, 2017 at 1:57 AM, Martell Malone via cfe-commits < > cfe-commits@lists.llvm.org> wrote:

r313088 - Revert "[Driver] MinGW: Remove custom linker detection"

2017-09-12 Thread Martell Malone via cfe-commits
Author: martell Date: Tue Sep 12 15:58:12 2017 New Revision: 313088 URL: http://llvm.org/viewvc/llvm-project?rev=313088=rev Log: Revert "[Driver] MinGW: Remove custom linker detection" This reverts rL313082 Modified: cfe/trunk/lib/Driver/ToolChains/MinGW.cpp

Re: r313082 - [Driver] MinGW: Remove custom linker detection

2017-09-12 Thread Martell Malone via cfe-commits
e Edwards > > On Tue, Sep 12, 2017 at 3:14 PM, Martell Malone via cfe-commits < > cfe-commits@lists.llvm.org> wrote: > >> Author: martell >> Date: Tue Sep 12 15:14:18 2017 >> New Revision: 313082 >> >> URL: http://llvm.org/viewvc/llvm-project?rev=313082=

Re: r313082 - [Driver] MinGW: Remove custom linker detection

2017-09-12 Thread Martell Malone via cfe-commits
Would you please have a look at: >>> http://green.lab.llvm.org/green/job/clang-stage1-cmake-RA-in >>> cremental/42380/ >>> >>> it appears this commit is causing some tests to fail on our bots. Any >>> assistance you may provide in getting the bots back green

Re: r313088 - Revert "[Driver] MinGW: Remove custom linker detection"

2017-09-12 Thread Martell Malone via cfe-commits
Will do in future, thanks Rui. On Wed, Sep 13, 2017 at 12:03 AM, Rui Ueyama <r...@google.com> wrote: > When you revert a commit, please explain why you are reverting it in the > commit message. > > On Tue, Sep 12, 2017 at 3:58 PM, Martell Malone via cfe-commits < > cf

r313082 - [Driver] MinGW: Remove custom linker detection

2017-09-12 Thread Martell Malone via cfe-commits
Author: martell Date: Tue Sep 12 15:14:18 2017 New Revision: 313082 URL: http://llvm.org/viewvc/llvm-project?rev=313082=rev Log: [Driver] MinGW: Remove custom linker detection In rL289668 the ability to specify the default linker at compile time was added but because the MinGW driver used custom

r319295 - Revert "Toolchain: Normalize dwarf, sjlj and seh eh"

2017-11-28 Thread Martell Malone via cfe-commits
Author: martell Date: Tue Nov 28 22:51:27 2017 New Revision: 319295 URL: http://llvm.org/viewvc/llvm-project?rev=319295=rev Log: Revert "Toolchain: Normalize dwarf, sjlj and seh eh" This reverts rL319294. The windows sanitizer does not like seh on x86. Will re apply with None type for x86

r319294 - Toolchain: Normalize dwarf, sjlj and seh eh

2017-11-28 Thread Martell Malone via cfe-commits
Author: martell Date: Tue Nov 28 22:25:13 2017 New Revision: 319294 URL: http://llvm.org/viewvc/llvm-project?rev=319294=rev Log: Toolchain: Normalize dwarf, sjlj and seh eh adds -fseh-exceptions and -fdwarf-exceptions flags clang will check if the user has specified an exception model flag, in

r319297 - Toolchain: Normalize dwarf, sjlj and seh eh

2017-11-28 Thread Martell Malone via cfe-commits
Author: martell Date: Tue Nov 28 23:25:12 2017 New Revision: 319297 URL: http://llvm.org/viewvc/llvm-project?rev=319297=rev Log: Toolchain: Normalize dwarf, sjlj and seh eh This is a re-apply of r319294. adds -fseh-exceptions and -fdwarf-exceptions flags clang will check if the user has

Re: r319297 - Toolchain: Normalize dwarf, sjlj and seh eh

2017-11-29 Thread Martell Malone via cfe-commits
ity_seh0 > Surface.obj : error LNK2001: unresolved external symbol > __gxx_personality_seh0 > > I'll dig in a bit to see if there's an easy fix. > > On Tue, Nov 28, 2017 at 11:25 PM, Martell Malone via cfe-commits < > cfe-commits@lists.llvm.org> wrote: > >> Author: martell >&g

Re: r319297 - Toolchain: Normalize dwarf, sjlj and seh eh

2017-12-01 Thread Martell Malone via cfe-commits
> >>> wrote: >>> On Wed, 29 Nov 2017, Martell Malone via cfe-commits wrote: >>> Thanks for letting me know Reid. >>> I’ll in work and won’t be able to access the repo >>> until lunch time. (~3 >>>

r318736 - [MS] add an init test for thumbv7-windows-msvc

2017-11-21 Thread Martell Malone via cfe-commits
Author: martell Date: Tue Nov 21 00:09:59 2017 New Revision: 318736 URL: http://llvm.org/viewvc/llvm-project?rev=318736=rev Log: [MS] add an init test for thumbv7-windows-msvc Modified: cfe/trunk/test/Preprocessor/init.c Modified: cfe/trunk/test/Preprocessor/init.c URL:

r318755 - [MINGW] normalize WIN32 macros

2017-11-21 Thread Martell Malone via cfe-commits
Author: martell Date: Tue Nov 21 03:28:29 2017 New Revision: 318755 URL: http://llvm.org/viewvc/llvm-project?rev=318755=rev Log: [MINGW] normalize WIN32 macros move _WIN64 and _WIN32 defines to lib/Basic/Targets/OSTargets.h move WIN32, WIN64 and __MINGW64__ to addMinGWDefines fixes __MINGW64__

r318608 - [Driver] add initial support for alpine linux

2017-11-18 Thread Martell Malone via cfe-commits
Author: martell Date: Sat Nov 18 16:08:12 2017 New Revision: 318608 URL: http://llvm.org/viewvc/llvm-project?rev=318608=rev Log: [Driver] add initial support for alpine linux set -pie as default for musl linux targets add detection of alpine linux append appropriate compile flags for alpine

r317397 - [Driver] MinGW: Remove custom linker detection

2017-11-03 Thread Martell Malone via cfe-commits
Author: martell Date: Fri Nov 3 19:07:59 2017 New Revision: 317397 URL: http://llvm.org/viewvc/llvm-project?rev=317397=rev Log: [Driver] MinGW: Remove custom linker detection This is a re-apply of rL313082 which was reverted in rL313088 In rL289668 the ability to specify the default linker at

r317398 - CodeGenCXX: no default dllimport storage for mingw

2017-11-03 Thread Martell Malone via cfe-commits
Author: martell Date: Fri Nov 3 19:15:49 2017 New Revision: 317398 URL: http://llvm.org/viewvc/llvm-project?rev=317398=rev Log: CodeGenCXX: no default dllimport storage for mingw GNU frontends don't have options like /MT, /MD This fixes a few link error regressions with libc++ and libc++abi