Hi, if it would be very important to support that old version of NASM you could just write "raw" assembler by putting the needed instructions as word's within the executable code.
changing the min requirement seems more proper though :) Thanks Michael On Thu, Jul 14, 2016 at 4:28 PM, Gao, Liming <liming....@intel.com> wrote: > Laszlo: > Sorry. I didn't mention my version. I use 2.12.01 to verify all changes. > In 2.12, it adds support of Codeview version 8 (cv8) debug format for win32 > and win64 formats in the COFF backend. We need this feature to get cv8 > debug. But, this version will report the error message: nasm: fatal: > assertion !"relocation for unregistered symbol" failed at > output/codeview.c:420. So, I suggest to update comments to NASM 2.12.01. > > Thanks > Liming > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Laszlo Ersek > Sent: Thursday, July 14, 2016 8:04 PM > To: Gao, Liming <liming....@intel.com>; Justen, Jordan L < > jordan.l.jus...@intel.com>; Kinney, Michael D <michael.d.kin...@intel.com> > Cc: edk2-devel-01 <edk2-de...@ml01.01.org> > Subject: Re: [edk2] minimum NASM version > > On 07/14/16 13:19, Laszlo Ersek wrote: > > > It seems that before NASM 2.10, there's simply no way to access segment > > registers in 64-bit mode. I propose that we upgrade the following > > comment in BaseTools: > > > >> diff --git a/BaseTools/Conf/tools_def.template > b/BaseTools/Conf/tools_def.template > >> index 2065fa34998f..db61a37b6afd 100644 > >> --- a/BaseTools/Conf/tools_def.template > >> +++ b/BaseTools/Conf/tools_def.template > >> @@ -708,7 +708,7 @@ DEFINE SOURCERY_CYGWIN_TOOLS = /cygdrive/c/Program > Files/CodeSourcery/Sourcery G > >> # > >> # Other Supported Tools > >> # ===================== > >> -# NASM 2.07 or later http://www.nasm.us/ > >> +# NASM 2.10 or later http://www.nasm.us/ > >> # > >> > #################################################################################### > >> > #################################################################################### > > > > plus wherever the minimum NASM version is documented on the Wiki. > > * In support of the above suggestion: in my Fedora 13 guest (set up for > GCC44 build testing), I could rebuild the following NASM package without > any problems, from the source RPM: > > nasm-2.10.09-1.fc21 > http://koji.fedoraproject.org/koji/buildinfo?buildID=463939 > > With this package installed, the build completed fine. > > * Also in support: the "oldstable" Debian release (code name "wheezy") > ships nasm 2.10 as well: > > https://packages.debian.org/search?keywords=nasm > > Thanks > Laszlo > _______________________________________________ > edk2-devel mailing list > edk2-devel@lists.01.org<mailto:edk2-devel@lists.01.org> > https://lists.01.org/mailman/listinfo/edk2-devel > _______________________________________________ > edk2-devel mailing list > edk2-devel@lists.01.org > https://lists.01.org/mailman/listinfo/edk2-devel > _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
