Re: [PATCH][2/2] Add store merging unit tests

[Jeff Law]

On 11/08/2016 05:03 AM, Kyrill Tkachov wrote:

Hi all,

This patch adds the plumbing for unit testing of the store merging pass.
It also adds some initial tests of some of the helpers used in
encode_tree_to_bitpos
to manipulate byte arrays. They caught an off-by-one error bug that is
fixed in patch [1/2].

Bootstrapped and tested on x86_64 and aarch64.

Ok for trunk?

Thanks,
Kyrill

2016-11-08  Kyrylo Tkachov  <kyrylo.tkac...@arm.com>

    * gimple-ssa-store-merging.c: Include selftest.h
    (verify_array_eq): New function.
    (verify_shift_bytes_in_array): Likewise.
    (verify_shift_bytes_in_array_right): Likewise.
    (verify_clear_bit_region): Likewise.
    (verify_clear_bit_region_be): Likewise.
    (store_merging_c_tests): Likewise.
    * selftest.h (store_merging_c_tests): Declare prototype.
    * selftest-run-tests.c (selftest::run_tests): Run
    store_merging_c_tests.

OK.
jeff