Revert "orcprogram-c: avoid running out of bounds of varnames"

This reverts commit 951091788b8496868bf86b2f853f5214659e782a.

This isn't a proper fix and the risk of bringing in a regression is too
high for the small and incomplete improvement.

configure: Only check for Android's liblog on Android targets

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

configure: Make -Bsymbolic check work with clang.

Update the -Bsymbolic check with the version glib has. This version
works with clang.

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

orcprogram-c: avoid running out of bounds of varnames

These two for loops use varnames[i], since varname has 48 items avoid
running out of bounds by running the for loop up to that limit.

CID 1147004

doc: update from entropywave locations to gstreamer

The entropywave website isn't even up any longer.
Docs have been uploaded to gstreamer website too now.

doc: update generated opcode tables

tools: remove unused test.orc

build: fix detection of AMD Kn host cpu

Not that anyone noticed in all these years, so
the check is probably bogus.

executor: fix load of parameters smaller than 64 bits

Parameters less than 64 bits are passed in just one param register.
Fixes crashes on ldresnearb and friends in emulated code.

compiler: also prefer the backup function when no target

If we don't have a compiler target (like for ppc64), prefer to use the
backup function instead of doing full emulation.

test-limits: improve test without target

When there is no target available, the unit test would fail with a
non-fatal error (because emulation would be done). The non-fatal error
would however not be overwritten by the real error when we exceed the
limits.

First relax the first compilation test to check for FATAL errors. Then
reset the program (and clear any non-fatal errors) before triggering
the next error.

Back to development

Release 0.4.24

parse: increase array size

Increase the size of the array to hold our variable name and use
snprintf to avoid overflows.

program: only reuse constants of the same size and value

When reusing an existing constant, make sure not only the value but also
the size is the same. We then also need to make a name based on the size
and value so that we can store the same value for different sizes.

Fixes problem with the bayer functions that used 255 as a 16 and 8 bit
constant. This was not detected in the unit test because both the backup
and asm functions do the same error, but it could be seen when a new
backup function was checked against a older ORC.

testsuite: fix reading .orc files with win32 line endings on windows

Just read them in binary form with win32 line endings and
parse them to the orc parser as-is now that the parser
can handle win32 line endings.

testsuite: add parsing test case for win32 line endings

Should handle \r\n as well as \n instead of relying on
the file reader to translate it.

orcc: fix reading .orc files with win32 line endings on windows

We used to open the files in text/translation mode, which
makes fread() convert all windows-style line endings (\r\n)
to unix style line endings (\n) under the hood. However, the
file size we get from the seek is apparently still the original
binary size, and then we just get less data because the \r have
been removed. This means we put our zero terminator in the wrong
place, and there might be some junk between the end of the
text data and the zero terminator.

Fix this by reading the file in binary mode (who knows what
else the text mode might mess up or trip over, e.g. unicode
chars in comments or such) and pass it to the orc parser
as-is now that the parser can handle win32 line endings.

Spotted by Ray Molenkamp.

orcparse: handle win32 \r\n line endings

tests: use AM_TESTS_ENVIRONMENT

So test environment gets used also with the
automake test runner. The code has fallbacks
so this doesn't actually fix anything in
particular.

test: protect against out-of-bounds

Same as commit 8bc018cbbe25521f91ecba0fc98185d2eaac650d

CID #1146994
CID #1146995

test: protect against going out of bounds

Conditions should never happen but protect against going out of bounds in the
args array.

CID #1146993

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