orc: remove empty and unused orcrules-x86.c

orc: move unusable orc_code_chunk_free() declaration to internal header

Nothing in any public header ever returned or used an OrcCodeChunk.

autotools: control symbol visibility via -fvisibility

All symbols are hidden by default unless exported
explicitly via our public API decorators.

meson: fix symbol export with MSVC and use -fvisibility elsewhere

Use newly-added ORC_API decorators to export symbols explicitly.

Get rid of orc.map file and the ORC_EXPORT stuff that never did
anything anyway (define was unused).

orc: decorate public symbols with ORC_API

We'll use that later to control symbol visibility and
exported symbols.

orc-test: decorate public symbols with ORC_TEST_API

We'll use that later to control symbol visibility and
exported symbols.

orcc: add --decorator command line argument

So we can add things like FOO_API if the functions are
supposed to be public, for example.

meson: use -Bsymbolic-functions when linking if available

cc.has_linker_argument() requires Meson 0.46, so guard
against older Meson versions for now.

orc: make more functions static

All of these are only used internally and are
not in any header files, but were still exported
by accident because they start with orc_*.

gtk-doc: get rid of templates

gtkdoc-mktmpl was removed in recent gtk-doc versions.

Update gtk-doc.make to 1.27, but modify to generate
xml/version.entities (i.e. add copy in right dir).

All a bit cargo-culted, but seems to work.

orcmips: only enable if the DSPr2 ASE is present

The mips target uses various instructions that a part of the MIPS DSPr2
ASE which is not present on all processors. The easiest solution is to
gate the target on the presense of DSPr2 at runtime.

Unfortunately there isn't a "nice" way to detect if DSPr2 is present, so
resort to parsing /proc/cpuinfo to get the information.

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

orcutils: refactor /proc/cpuinfo reading code from orccpu-arm.c

This code will later be used by mips target, so it makes sense
to use a common function instead of duplicating the code.

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

configure: don't check for functions that aren't used anywhere

And remove some unneeded includes.

meson: fix mips cpu_family test

The cpu family "mipsel" does not exist in meson so the current test to
enable the HAVE_MIPSEL macro is always false. The real cpu family is
"mips" but this also matches big endian platforms so also check that the
endian is little.

https://bugzilla.gnome.org/attachment.cgi?id=370595

orccpu-arm: Allow 'cpuinfo' fallback on non-android

Reading the cpu flags from cpuinfo is also useful on non-android
ARM platforms which might have mechanisms that prevent /proc/*/auxv
from being read.

meson: Always build both shared and static libraries

This reuses objects from the shared library for the static library, so
it should not affect the build time at all.

orc_dep and orc_test_dep will still contain the library selected by
--default-library

Build liborc-test also as a properly versioned library with meson

Add meson configuration for disabling tests and gtk-doc docs

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

Allow building shared/static library based on configuration

Before only a shared library was possible

Back to development

Release 0.4.28

orcutils: Use compiler offsetof() when available

The way we were calculating the offset of a structure entry was
undefined behaviour.

Instead, use 'offsetof()' (defined in C89) when available

orccpu-x86: Fix undefined behaviour compilation issue

runtime error: left shift of 1 by 31 places cannot be represented in type 'int'

configure.ac: Print configuration after configuration

Makes it easier to spot/review detected flags

orc: Add a configure option to disable tests

For targets where it doesn't make sense.

Note that this will disable the build of orc-bugreport since it
depends on the the orc_test helper library

tesuite: Fix some leaks

orcprogram-c.c: Don't cast constant

Clang compilers would complain with constant-conversion error (trying
to fit a 'int' into another type)

This was introduced to fix a warning in gcc ... which doesn't happen

orc: Cast maximum constants

Avoids constant-conversion warnings as such:

/src/orc/orc/orcemulateopcodes.c:158:13: error: implicit conversion from 'int' to 'orc_int8' (aka 'signed char') changes value from 255 to -1 [-Werror,-Wconstant-conversion]
    var34 = ORC_CLAMP_UB((orc_uint8)var32 + (orc_uint8)var33);
          ~ ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/src/orc/orc/orcemulateopcodes.c:29:48: note: expanded from macro 'ORC_CLAMP_UB'
                                               ^~~~~~~~~~
/src/orc/orc/orcemulateopcodes.c:18:20: note: expanded from macro 'ORC_UB_MAX'
                   ^~~
/src/orc/orc/orcemulateopcodes.c:12:55: note: expanded from macro 'ORC_CLAMP'
                                                      ^

Dist meson files with autotools make dist

Back to development

Release 0.4.27

orcc: make coverity happy

Use typedefed enum so static analysis tools like coverity can know
we covered all possibilities in switch/case statements. CID 1413336.

orc: preserve non volatile sse registers

required for MSVC
https://msdn.microsoft.com/en-us/library/9z1stfyw.aspx

orc already marks these register in compiler->save_regs,
but the actual saving and restoring was not done

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

meson: Only use --version-script if not using MSVC

meson: use find_library() on compiler object

The old stand-alone find_library() was deprecated.

meson: add meson build

Includes contributions from Nirbheek Chauhan <nirbheek@centricular.com>

orcx86: Don't hard-code register size to zero in orc_x86_emit_*() functions

Instead use the size passed as argument. Fixes segmentation fault on QNX.

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

orcx86: use HAVE_AMD64 and HAVE_I386 defines everywhere

Some places were checking for __amd64__ and __i386__ explicitly,
but MSVC doesn't define __amd64__ in 64-bit mode, so use the
define from config.h instead. Fixes incorrect asm generation
on 64-bit Windows

Back to development

Release 0.4.26

pkgconfig: add orc-test path to the -uninstalled .pc file

Expose orc-test library in the uninstalled pkgconfig file.

use 64bit arithmetic to increment the stride

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

orcx86: Fix generation of ModR/M / SIB bytes for the EBP, R12, R13 registers

See http://wiki.osdev.org/X86-64_Instruction_Encoding#ModR.2FM

ESP and R12 are to be handled exactly the same, that is the addressing is
done with SIB instead of directly.

EBP and R13 have a special case for Mod 00, where they're expecting RIP/EIP
plus 32 bit displacement. If we handle them like all other registers, we would
instead of addressing offset 0, address something relative to the instruction
pointer and make part of the following instruction the 32 bit displacement.

Instead of using Mod 00 for offset 0 like for all other registers, we use Mod
01 and a 8 bit displacement of 0.

On Windows this rather low-level bug caused problems at a rather
high-level in the ORC video conversion code, where a movdqu was eating the
following paddw instruction and tried to load data from a rather random memory
location.

For whatever reason this didn't cause any other big problems in other ORC
generated code, and it also only seemed to have caused problems on Windows.

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

tests: fix a warning spotted when building in osx

orctest.c:730:19: error: logical not is only applied to the left hand side of this comparison
       if (!float_compare (dest_emul[l-ORC_VAR_D1], dest_exec[l-ORC_VAR_D1], i, j) != 0) {

It seems some sort of typo. Fixing by removing the potentially leftover condition.

Fix test_parse with no executable backend

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

Back to development

Release 0.4.25

configure: fix bug report url

orcc: check version of liborc to make sure the right lib is being used

Interesting compilation failures might otherwise ensue.

Add orc_version_string()

uninstalled.pc: add support for non libtool build systems

Currently the .la path is provided which requires to use libtool as
mentioned in the GStreamer manual section-helloworld-compilerun.html.
It is fine as long as the application is built using libtool.

So currently it is not possible to compile a GStreamer application
within gst-uninstalled with CMake or other build system different
than autotools.

This patch allows to do the following in gst-uninstalled env:
gcc test.c -o test $(pkg-config --cflags --libs orc-uninstalled)
Previously it required to prepend libtool --mode=link

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

orcprogram-c: check array bound in get_varname_stride()

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

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.