[Qemu-devel] Re: [Qemu-commits] [COMMIT 968a40f] microblaze: Trap on unaligned data accesses.

[Blue Swirl]

On Thu, Sep 3, 2009 at 2:11 PM, Anthony Liguori<address@hidden> wrote:
> From: Edgar E. Iglesias <address@hidden>

> +void helper_memalign(uint32_t addr, uint32_t dr, uint32_t wr, uint32_t size)
> +{
> +    uint32_t mask;
> +
> +    switch (size) {
> +        case 4: mask = 3; break;
> +        case 2: mask = 1; break;
> +        default:
> +        case 1: mask = 0; break;
> +    }

For additional performance (and to avoid the check for mask == 0 case
;-)), you could perform most of the math above during translation.

Sparc32 actually avoids most of these explicit tests, but I still
can't understand Fabrice's magic that made it possible.