preproc: %ifenv should require environment variable name to exist

Otherwise naked %ifenv/%endif passes without a notice.

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

BR3414012: Make %if[n]def to handle tail spaces

In case if there a trailing spaces after expansion
%if[n]def might produce a false alarms on macro id
being expected.

Note it also makes the constructions like bare

%ifdef
%endif

invalid, while before they were passed well without
a notice.

Reported-by: KO Myung-Hun
Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

assemble: Drop redundant variable

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

BR3411280: Fix incorrect compiler option for OS/2 target in Open Watcom makefile

For OS2 target the -bt option should be set accordingly.

Signed-off-by: Jiri Malak <malak.jiri@gmail.com>
Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

insns.dat: Fixup VGATHERx instructions

As being spotted by nasm64developer the memory
operands size is incorrect. Fix it.

Reported-by: nasm64developer
Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

Add AVX2 test file

nasm64developer kindly provided a testfile converted
to nasm format.

http://sourceware.org/ml/binutils/2011-06/msg00150.html

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

A couple of simplifications to assemble.c

 - GEN_SIB and GEN_MODRM helpers added
 - a number of tabs vs space fixs
 - more use of is_class() helper

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

parser: No need to split line if it a bit longer than 80 chars

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

Add IS_SREG and IS_FSGS helpers

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

A few style fixups in parser.c

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

elf64: Make linelist to look the same as elf32

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

elf32: Drop holes in linelist structure

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

outobj: Reorder Segment members to eliminate holes

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

outieee: Reorder ieeeSection members to eliminate holes

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

Drop empty line and bracket

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

Style tuneup of nasm.h

 - a number of tabs to space conversion
 - redundant spaces removed
 - structure elements alignment

No functional changes.

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

disasm.c: ESP/RSP cannot be indicies, but XMM4/YMM4 can be

If we have XMM4/YMM4 as a VSIB, it does not mean no index; that is
only for scalar encoding.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

assemble.c: remove stray debugging code

My bad for checking this in at all.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

insns, avx2: A couple of upper-case to lower-case conversion

Just to be solid in style

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

insns, avx2: A typo in VPERMPD

The second VPERMD should be VPERMPD actually.

Thanks to nasm64developer for gas test file provided
which allowed to reveal this issue.

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

assemble.c: Comment out debug printing

Probably we need some kind of pr_debug or
something like that instead.

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

BR3385573: Some AVX2 instructions fixups

A few instruction templates for AVX2 set were wrong.

Reported-by: Agner Fog
Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

insns: Change VPERMPQ to VPERMQ

Was a typo

Reported-by: Agner Fog
Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

preproc: Drop emitting() rudimanet

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

doc: update documentation for pending 2.10 release

NASM 2.10rc8

Implement insns.dat in human readable form

 I converted almost all instructions in insns.dat (version
 7a6f97869840fc0b1b8677bd77c74754c5f024aa) to the more
 readable format that insns.pl has supported for years.

 I also made some changes to insns.pl. You can verify that the
 new insns.dat and insns.pl produce byte-identical output to
 the old insns.dat and insns.pl, so I think that this change
 is safe to check in, even though it is a large change to
 insns.dat.

The changes to insns.pl are:

 * fixed a bug: ib,u was not recognized
 * added support for a second immediate argument called "j" for
   instructions like ENTER imm,imm
 * added a "+r" syntax for \10..\13

[gorcunov: insns files remains the same, great job anonymous!]

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

NASM 2.10.rc7

Intel's AVX2 instructions should be covered now.

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

insns: A final pile of AVX2 instructions

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

insns: A few additional AVX2 templates

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

insns: One more small snippet of AVX2

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

insns: One more slab of AVX2 instructions

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

insns: Add a slab of AVX2 instructions

Not all are covered yet, but still a step
forward I think.

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

insns.dat: Move exsiting AVX2 insns to a separate section

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

preproc: Align members in structures

For easier reading

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

preproc: Reorder ExpInv to drop padding

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

preproc: Reorder SMacro members to eliminate padding

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

nasm.h: Reorder tokenval members to drop padding

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

Some cleanup on insns.h

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

Drop unused 'opt' from nasm_opt_val

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

elf64: Drop unused 'zero' variable

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

Drop unused 'type' from gencode

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

Fix saa_writeaddr for big-endian case

saa_writeaddr on big-endian archs didn't write converted
argument, fix it.

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

Drop unused 'arg' variable

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

Merge branch 'nasm-2.09.xx'

Conflicts:
version

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

NASM-2.09.10

- Fix nasty problem with uninstaller in case of registry
  keys corruption, which might lead to arbitrary files
  removal.

  Just to be sure the proper files and directories are to
  be removed a few additional questions are brought into
  deinstallation procedure.

  The former issue reported here

http://forum.nasm.us/index.php?topic=1013.0

  Note: no functional changes are done inside nasm code,
  critical fixes will be ported to 2.09.xx branch, but
  mostly the (more or less) active development and various
  bug fixes are concentrated around 2.10 series.

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

doc: Describe changes for nasm-2.09.10

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

nsis: Remove desktop bat link as well

In case of installation procedure being used in
a per-user fasion the display link to a bat file
remain untouched at uninstallation time. Fix it.

I hope this time all things go smooth.

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

nsis: Inform a user about files being deleted

Since a user might have some weird access rules on NTFS
such as allow to write files but not to delete them it
might happen that uninstallation procedure fails at
deleting files but deleting registry keys still success,
and in result a user might try to uninstall nasm for second
time with registry keys corrupted.

So prompt a user with files being deleted so he can double
check what is going on.

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

nsis: Make sure the registry keys exist

It was reported on nasm forum (http://forum.nasm.us/index.php?topic=1013.0)
that if uninstallation procedure get stopped (for any reason) in a middle
leading to read empty registry values an attempt to remove arbitrary
files might happen. So make sure there are some sane values in registry.

N.B: People, if you find some bug in nasm, don't write to forum, file
bug directly in bugzilla, if it's not a bug we simply close it, otherwise
we might miss serious problems like this one!

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

nsis: Remove desktop bat link as well

In case of installation procedure being used in
a per-user fasion the display link to a bat file
remain untouched at uninstallation time. Fix it.

I hope this time all things go smooth.

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

nsis: Inform a user about files being deleted

Since a user might have some weird access rules on NTFS
such as allow to write files but not to delete them it
might happen that uninstallation procedure fails at
deleting files but deleting registry keys still success,
and in result a user might try to uninstall nasm for second
time with registry keys corrupted.

So prompt a user with files being deleted so he can double
check what is going on.

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

A few more AVX2 spec instructions

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

Use a normal quad-case for valueless /is4

When we don't have an immediate for the i-field in /is4, then use a
normal quad-bytecode encoding for it to save some small amount of
space and re-use existing machinery.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

Remove support for DREX encoding

The DREX encoding never hit production silicon, and has been replaced
by VEX/XOP encoding, so remove support for it.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

insns.dat: Clean up and fix the BMI instruction patterns

Clean up the formatting of the BMI instruction patterns, and fix:

a) X64,FUTURE is wrong - it needs to be LONG,FUTURE
b) Fix the BLSI, BLSMSK, BLSR instruction patterns
c) Use a bracket pattern for TZCNT

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

insns: Fix up RORX template

Missed 64 bit case.

Signed-off-by: Jasper Neuman <jasper.neumann@web.de>
Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

insns.dat: Add some BMI1 and BMI2 instructions

Signed-off-by: Jasper Neuman <jasper.neumann@web.de>
Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

nsis: Make sure the registry keys exist

It was reported on nasm forum (http://forum.nasm.us/index.php?topic=1013.0)
that if uninstallation procedure get stopped (for any reason) in a middle
leading to read empty registry values an attempt to remove arbitrary
files might happen. So make sure there are some sane values in registry.

N.B: People, if you find some bug in nasm, don't write to forum, file
bug directly in bugzilla, if it's not a bug we simply close it, otherwise
we might miss serious problems like this one!

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

insns.h: Add BMI1 and BMI2 flags for further usage

Signed-off-by: Jasper Neuman <jasper.neumann@web.de>
Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

preproc: Drop useless assignments

new_ExpDef does zalloc'ate memory so no need
to assign zeros again.

Same time it should fix MSC complains on NULL assignment
on boolean variable.

Reported-by: Jasper Neuman <jasper.neumann@web.de>
Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

Merge branch 'nasm-2.09.xx'

Conflicts:
version

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

NASM 2.09.09

Just a few fixes only but serious enough to yield
new stable series

 - Unitialized memory parsing in bin format section attibutes.
 - MachO NULL dereference

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

doc: Describe changes for stable version

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

BR3282788: Fix 64-bit Mach-O bug that crashes NASM due to NULL symbols

output/outbin.c: initialize section attribs upon creation

Basically it's backport of commits

11db774a151e9d895fa05f980563a5cafb0f306a
3bc3ff2fb685a645698f9db9cfc903df30e4e555
c13deef255b621ace2130adf55530f3364a40458
e3f47806658de042af0eaccb1cc7896be388b397

They were missed to back-merge in a first place.

Reported-by: Keith Kanios <keith@kanios.net>
Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

preproc: Move Preproc type to preproc_ops structure

There is no need to hide this structure into a type.
The former preproc_ops is a way more descriptive.

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

preproc: Refactor smacro paramters expansion

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

preproc: Add smacro paramter helpers

I belive this should simply code a bit.

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

preproc: Alignment in loop

Just for easier reading

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

insns: Mark AVX2 instructions as FUTURE

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

preproc: Finally drop context-through search

2.09 series was the last one we support context-thru search
(and we were issuing a warning about that) so drop all-context
from get_ctx() routine.

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

preproc: nasm_free is safe against NULL argument

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

Revert "BR3288901: Relax concat rules in preprocessor code"

This reverts commit cb00cd1ba7f07f60f0a94c43cfc4fa7b01e2ff95.

As Victor pointed out some tests do not pass with this commit
so revert it until things get fixed.

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

preproc: Add trace point into paste_tokens

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

preproc: Some more tracing calls

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

preproc: Missed double charp in nasm_trace

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

insns: A few more AVX2 instructions

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

insns: Add VPERMD instruction

We somehow missed it.

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

test: Add movd.asm

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

insns: Allow MOVD xmmreg,rm32 to be used in 32bit mode

Reported-by: Keith Kanios <keith@kanios.net>
Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

preproc: Add tokenization tracing

It's a bit more than that, also TRACE=1 make
flag added to run this facility on and off
at compiling time.

Debug feature only, doesn't affect regular users.

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

preproc.c: Get rid of a few tabs and update year

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

preproc.c: Use list_reverse helper

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

nasmlib: Add list_reverse helper

There is no way to make it as a helper function
for a while, we need to unify list processing
in general (ie to use abstract list type).

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

nasmlib: Add list_last helper

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

Move numvalue herleper into nasmlib.h

No need to duplicate implementation.

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

preproc: Drop unused 'mtok' variable

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

nassm.c: Use evaluate for section alignment

This allow us to write the whole expressions
on section alignments, such as

align 0xa+6

or whatever math. Should be a way more convenient
than hardnumbers scheme we had.

Reported-by: Frank Kotler <fbkotler@zytor.com>
Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

BR3288901: Relax concat rules in preprocessor code

We simply allow the following terminals to be concat'ed
if they are written without space or any other separator
inbetween.

        a := id | preproc-id | number | float | other
        b := id | preproc-id | number | float | other

        if match(a,b):
                s := concat(a,b)
                re-tokenize(s)

Basically it means it's up to code author to write
preproc code a way the sane production appears.

Some notes.

1) We don't concat strings.

2) The 'weirdpaste' test fails now because with relaxed
   rules it works as needed and was borken before.

   The lacmus snippet is

        %define N 1e%++%+ 5
        dd N, 1e+5

Previously the output was

        dd 1e+%+ 5, 1e+5

which is wrong since we have explicit concat here
with %+ operator. The new code production is correct
and looks like

        dd 1e+5, 1e+5

as expected.

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

insns: Mark VGATHERDPD as AVX2 instructions

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

Add IF_AVX2 flag

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

Merge branch 'nasm-2.09.xx'

rdoff: add missing dependency in Makefile

Add back a dependency which was accidentally dropped for the previous
build bug fix.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

Merge branch 'nasm-2.09.xx'

Makefile: hopefully fix rdoff parallel build problems

The strange dependency on nasmlib.o causes serious issues in parallel
build; hopefully fix it.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

Implement the VGATHERP instruction

As an initial test of the VSIB handling, implement the VGATHERP
instruction.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

Add support for VSIB instructions

Add support for VSIB instructions, which use vector registers as the
index registers in an EA.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

NASM 2.10rc6

spec: update to match requirements of Fedora 14+

"PreReq" is no longer accepted as an alias for "Requires".  Sigh.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>