On 10 January 2014 19:05, Richard Henderson <r...@twiddle.net> wrote:
> On 01/10/2014 09:12 AM, Peter Maydell wrote:
>> static void disas_data_proc_simd(DisasContext *s, uint32_t insn)
>> {
>> /* Note that this is called with all non-FP cases from
>> * table C3-6 so it must UNDEF for entries not specifically
>> * allocated to instructions in that table.
>> */
>> - unsupported_encoding(s, insn);
>> + AArch64DecodeFn *fn = lookup_disas_fn(&data_proc_simd[0], insn);
>> + if (fn) {
>> + (fn) (s, insn);
>
> Oh, do you want to CheckFPAdvSIMDEnabled64 here before calling fn?
> Otherwise that's the first thing I noticed missing from patch 4.
We don't currently check that for the FP insns either. Since it's a system
register check and will always pass for usermode emulation I was planning
to leave it for when I did system emulation and wired up the CPACR_EL1.
thanks
-- PMM
