orcc: Remove duplicate codepath

The code is identical regardless of the variable value

CID #1226451

https://bugzilla.gnome.org/show_bug.cgi?id=745830

testsuite: Put a limit to the memcpy test

Some systems have massive L3 cache that can exceed 64MB, which would
result in attempting to read/write about our allocated memory.

If we can deal with 64MB... there's a good chance we can handle above.

orccodemem: fix mmap leak on error path

Not that these seem to be freed at all on non error

Back to development

Release 0.4.23

neon: add support for all constants

Use mov and orr to construct all constants.

neon: add support for 64 bit parameters

neon: vtrn and vzip are undefined when src == dest

Make sure we use different src and dest registers for vtrn and vzip
because otherwise they are undefined.

orcc: allow setting custom backup function

Add a new .backup keyword that instructs the orc compiler to call our
custom backup function instead of generating one. This is interesting if
the generated backup function is slower than a plain C implementation.

Fix high-order bit of first register in VPUSH/VPOP generation

CID 1256553
CID 1256554

https://bugzilla.gnome.org/show_bug.cgi?id=727464

Preserve NEON/VFP registers across subroutines according to ARM PCS (5.1.2.1)

https://bugzilla.gnome.org/show_bug.cgi?id=727464

orcc: program-c: fix 64-bit parameter loading (loadpq) on big-endian systems

When passing 64-bit parameters through OrcExecutor, we
have to split them up into two 32-bit parameters for
backwards compatibility reasons. When generating C code,
make sure that we split up 64-bit parameters in the same
way as loadpq will read them back later. The lower 32 bits
should end up in params[ORC_VAR_D1+i] and the higher bits
should end up in params[ORC_VAR_T1+i]. The way it was done
so far, the higher/lower bits ended up swapped on big endian
systems, and then got deserialised in swapped order by loadpq.
This resulted in bogus parameters being used.

In particular, this broke the gstreamer volume element and
its unit tests on big endian systems when handling samples
in F64 format (i.e. doubles).

https://bugzilla.gnome.org/show_bug.cgi?id=739354

orc: include orcprogram only once

opcode: mark copy opcodes and don't emit mov for them

There is no need to emit a mov operation when the source and dest
registers of a copy operation are different, that's kindof the point of
the copy operation. Saves some useless movs whenever copyX is used.

compiler: use src register as dest2

Also try to use the src register as the second destination register for
opcodes with 2 results. Saves one unnecessary mov for splitX opcodes.

compiler: update max_used_temp_reg before use

It should not be smaller than min_temp_reg or we will fail to search for
a free register.

sse: improve splitXX on sse3

We can implement splitXX with byte shuffles on sse3

parse: cleanup

Be more elegant in parsing and passing the parameters to
orc_program_append_str_2().

test: add more tests

compiler: keep track of temp variables that contain param/const

Keep track of what const/param is loaded into a temp variable. We can
then see if the const is already loaded in a variable and simply reuse
it instead of using a new temp. We must also make sure we load constants
of different size in different temp variables.

Revert reuse constants

This reverts commit 2ece711c615d327be1e74bec4844186ced58b0ef.
This reverts commit 2ae41b98a35405dbfd35209e7d68428be77a2e62.
This reverts commit 106dacc2620126dedb99d2b981ba2e5a7d4591f9.
This reverts commit f6697e30681f46f6f3f632ddbbcab98107eec7e0.

These commits are more complicated than needed. A more simple solution
is possible by tracking the registers that hold a certain constant and
simply reusing that register when we can. We can then load the same
constant in different registers of different sizes if needed.

parse: make contants per size

When we automatically create constants from the opcode arguments, make
different constants per size and rewrite the program to use the newly
created constant.

parse: fix opcode arg size

executor: use the size of the opcode arg to get multiplier

When loading a constant, select the multiplier based on the size of the
opcode argument. This is safer because we have existing code that declares
the constants of the wrong size.

executor: fully expand const vars into the temp register

When loading a const in a register, always make it cover the full
register so that it containts enough values to be used in x2 and
x4 instructions.

parse: register constants with the right size

When we detect a constant while parsing a line, register it with the
same size as the argument of the opcode.

program-c: handle X2 and X4 with size < 2 variables

when the variable has a size < 2 it will have been declared as a simple
int8 or int16, when we use this in a X2 or X4 instruction, use the
variable directly instead of trying to use the x fields.

compiler: generate only 1 temp for constants

Generate only 1 temp variable for constants. We do this by keeping track
of what variable we used to replace the constant.

Previously, orc would allocate a new temp variable every time a constant
was used, this wastes registers and is not needed.

compiler: reuse _try_get_constant in _get_constant

opcodes: avoid undefined operations on signed ints

Cast to unsigned int when doing addition and subtraction that can
overflow.
Cast to unsigned int when doing the lower part of a multiplication.
Cast to unsigned int when doing left shifts into the sign bit.
Regenerate the emulation code. This should avoid multiple warnings
reported with clang and -fsanitize=undefined.

See https://bugzilla.gnome.org/show_bug.cgi?id=728738

compiler: do << and | on unsigned ints only

compiler: avoid nagative array index

For instructions that don't have a second variable, don't try to index
the variable array with a -1 index.

utils: convert to uin32 before doing |

In ORC_READ_UINT32_LE, convert each byte to uint32 before or-ing them
together. This avoids | with signed ints.

docs: remove old env variable

ORC_CPU_FLAGS has been removed quite some time ago

TMPDIR becomes the last option

Previously $TMPDIR and /tmp were the first options to try to use for
intermediate files. To prevent name collisions, now user specific dirs ($HOME
and $XDG_RUNTIME_DIR) are prefered over the user unspecific dirs ($TMP and
/tmp).

https://bugzilla.gnome.org/show_bug.cgi?id=735871

Back to development

Release 0.4.22

Add fallback CPU feature detection for Android

On Android, /proc/self/auxv might not be readable (except
when debuggable=true in the build, annoyingly), so do what
the android cpufeatures detection code does and fall back to
/proc/cpuinfo string matching.

Without this, release builds run really slowly, due to ORC always
doing emulation.

Remove declarations for deleted functions from orcutils.h

Detect android liblog and use android's log functions for debug output

orccompiler: free error message

orcarray: Use size_t for pointer arithmetics

Fixes these warnings on windows x86_64:

orcarray.c:80:33: error: cast from pointer to integer of different size [-Werror=pointer-to-int-cast]
orcarray.c:80:22: error: cast to pointer from integer of different size [-Werror=int-to-pointer-cast]

orccompiler: If compiling a program twice, first free the results of the previous compilation

orcc: Fix some memory leaks in the compiler

autogen: Honor NOCONFIGURE=1

See https://github.com/cgwalters/build-api

Back to development

Release 0.4.21

configure: Pass -version-info to libtool again

Back to development

Release 0.4.20

build: Modernize autotools build system

Add some checks on the number of variables per type

We want to ensure no more than, say, 8 constants are added
to a program. Adding more will violate pervasive assumptions
in the code, and may lead to various buffer overflows. By
trapping these add creation time, we prevent these issues
without cluttering the code with range checks. The user is
assumed non malicious here.

Add a test to check we can add up to and including the limit
for a type, but no more.

Update .gitignore

orccodemem: fix list corruption when splitting memory chunks in two

When splitting orc memory chunks in two, the 'prev' pointer of the
new memory chunk to be inserted was wrong, causing invalid memory
access and double frees later down the line.

https://bugzilla.gnome.org/show_bug.cgi?id=731227

orc: Close file after usage in the copied around file reading function

CID 1147011.

orcarray: Use fabs() instead of abs(), the latter is for integers

orcarray.c:254:15: error: using integer absolute value function 'abs' when
      argument is of floating point type [-Werror,-Wabsolute-value]
          if (abs(a[i] - b[i]) < MIN_NONDENORMAL_D) continue;

orcprogram-c: Convert shifts in swap opcodes to operations on unsigned integers

Shifting into the sign bit is undefined, so let's stop doing that.

memcpy_speed: Use unsigned integers, overflows are undefined for signed integers

Back to development

Release 0.4.19

orccodemem: mask group/other bits for temporary file

Ensures another user can't inject code into your process when
winning a race with the ORC temp file code.

Coverity 1147013

orcprogam-{mips,mmx,sse}: bail out if get_align_var fails

Coverity 1146979, 1146980, 1146977, 1146978, 1146973

orcprogram: bail out on error

Coverity 1146972

orcbytecode: Don't replace program name without freeing the old name if any

orcc: Fix indention of generated code

orc: Update autogenerated files

orcprogram-c: Fix swaplq C implementation

orcbytecodes: Fix filename of generated file

We don't want to override orcbytecode.h but orcbytecodes.h

orcarray: Fix compiler warning with gcc 4.9

In file included from ../orc/orcprogram.h:5:0,
                 from ../orc/orc.h:5,
                 from ../orc-test/orctest.h:5,
                 from orcarray.c:6:
../orc/orcutils.h:124:37: error: 'data' may be used uninitialized in this function [-Werror=maybe-uninitialized]
 #define ORC_PTR_OFFSET(ptr,offset) ((void *)(((unsigned char *)(ptr)) + (offset)))
                                     ^
orcarray.c:42:9: note: 'data' was declared here
   void *data;

memcpy_speed: Properly define ALIGN macro and fix memory leaks

With the previous definition we would use memory *before* our
allocated memory area.

bench10: Fix syntax errors

test.orc: Fix syntax errors

benchmorc: Fix memory leaks in test

orcparse: Guard against unknown tokens

orcprogram: Free init function name

tests: Fix various memory leaks in tests

orcprogram: Free OrcCode together with the program

orcprogram: Free type names of variables when freeing a program

bytecode_parse: Fix memory leaks in test

orcmips: Fix register range check

There are 32 possible registers and not 33

CID #1146987

orcexecutor: Don't use negative values on arrays

orc_program_find_var_by_name() can return -1 on errors, if that
happens, don't try to read/write in random memory :)

CID #1146974
CID #1146975
CID #1146976

orc: Remove unused variable

include_file was never used in these variants since they were
added.

CID #1146970
CID #1146969

testsuite: file leak fix

Coverity 1147011

Fix FILE leak

Coverity 1147011

orcrules-neon: Remove unused variable

docs: mulhsw, mulhsw shifts by 16, not 8

The documentation said that mulhsw and mulsuw was equivalent to:

    (a * b) >> 8

when the implementation actually shifts by 16 bits.

https://bugzilla.gnome.org/show_bug.cgi?id=723170

orcarm: OSX doesn't like comparisons against enum and int.

tools: fix memory leak in orcc

Coverity CID 1147542

testsuite: fix file descriptor leak in benchmorc

Coverity CID 1147543

Make it ANSI-C

Use /* */ instead of //
Define _GNU_SOURCE in the ORC_CFLAGS

autogen.sh: Allow calling from out of tree

This is similar to the fix that was done for gstreamer

https://bugzilla.gnome.org/show_bug.cgi?id=667664

configure: version back to unreleased

Release 0.4.18

orcbytecode: Fix parsing of 32 bit values from bytecode as well

The shift by 24 bits has to be casted as well, otherwise we
shift into the sign bit which causes undefined behaviour.

See https://bugzilla.gnome.org/show_bug.cgi?id=698520

orcbytecode: Fix parsing of 64 bit values from bytecode

The shift by 24 bits has to be casted already, otherwise we shift
into the sign bit which causes undefined behaviour.

Thanks to Edward Hervey for debugging this.

https://bugzilla.gnome.org/show_bug.cgi?id=698520

docs: add more opcode descriptions

docs: hook up generate_table tools

Add a makefile-target in docs to update the tables (for a release). Emit a
comment at the top of the generated files which tool produces them so that
people don't edit them manually.

accumulator: use the local variable 'var'

The code already had the local var, but barely used it.

tests: use the ORC_STATIC_OPCODE_FLOAT flag to simplify the tests

orcparse: fix copy and paste of error message