On 17 April 2017 at 05:20, Philippe Mathieu-Daudé <f4...@amsat.org> wrote:
> On 04/10/2017 02:13 PM, Peter Maydell wrote:
>> --- a/target/arm/translate.h
>> +++ b/target/arm/translate.h
>> @@ -31,6 +31,7 @@ typedef struct DisasContext {
>> bool vfp_enabled; /* FP enabled via FPSCR.EN */
>> int vec_len;
>> int vec_stride;
>> + bool handler; /* v7M Handler mode */
>
>
> What about a more descriptive name like 'is_v7m_handler_mode' or
> 'v7m_mode_handler'? It makes your next patch easier to understand:
Seems reasonable; I've made that change (but won't repost the
series as that's the only tweak).
thanks
-- PMM
