David, Thank you for the comments.
Please, take a look to updated webrev: http://cr.openjdk.java.net/~dsamersoff/JDK-8022617/webrev.03/ -Dmitry On 2013-08-29 04:41, David Holmes wrote: > Dmitry, > > I don't think this is something that should be handled at the configure > level. Hotspot compiler flags are handled in the hotspot makefiles. This > should be in gcc.make. > > BTW your changeset should include the generated-configure.sh file not > configure. And you would also need to regenerate and push the closed > generated-configure.sh file. > > David > > On 28/08/2013 11:50 PM, Dmitry Samersoff wrote: >> Hi Everyone, >> >> Please review small fix >> >> webrev: >> >> http://cr.openjdk.java.net/~dsamersoff/JDK-8022617/webrev.02/ >> >> CR: >> >> http://bugs.sun.com/view_bug.do?bug_id=8022617 >> >> >> Gory details: >> >> bsd_x86_64.s use macro to deal with OS X specific things. >> >> llvm-gcc preprocess .s and .S files and doesn't support .sx extension >> recommended by GNU for case insensitive filesystem. >> >> Other operating systems doesn't preprocess .s files, so bsd_x86_64.s >> couldn't be compiled on other bsd systems. >> >> This patch enforce of preprocessing of all assembly sources by >> command line options (-x assembler-with-cpp). >> >> -Dmitry >> -- Dmitry Samersoff Oracle Java development team, Saint Petersburg, Russia * I would love to change the world, but they won't give me the sources.