On Wed, Jul 11, 2018, 1:20 PM Hugo Beauzée-Luyssen <h...@beauzee.fr> wrote:

> On Wed, Jul 11, 2018, at 7:12 PM, Martin Storsjö wrote:
> > On Wed, 11 Jul 2018, Hugo Beauzée-Luyssen wrote:
> >
> > > RtlAddFunctionTable is only available on those. Moreover, so are the
> > > (P)RUNTIME_FUNCTION types
> >
> > No, it's available for armv7 as well (I just checked the msvc import
> > libs). So the correct logic probably is "HAVE_NON_I386" or something
> like
> > that. (We have a corresponding macro F_NON_I386 for .def files.)
> >
>
> Ok, I'll send an updated version :)
>
> > Also, to keep things simple, you could skip the build system part and
> just
> > use an ifdef in the source file; ar usually gives a small warning for
> > object files without symbols, but it shouldn't hurt much imo. But I
> won't
> > object to that part if you really want to have the condition in the
> > makefile.
> >
>
> I originally wanted to do so but disliked the "iso C forbids empty
> translation unit" warning, however if that's fine by you then I'll remove
> the configure/makefile part!
>

Builds should be warning free, from all tools and all configurations. This
is a quality regression I'm seeing across the entire project.

>
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Mingw-w64-public mailing list
Mingw-w64-public@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mingw-w64-public

Reply via email to