Hi Richard,

On 07/07/17 15:19, Richard Earnshaw (lists) wrote:
> 
> Hmm, I think that's because really this is a partial conversion.  It
> looks like doing this properly would involve moving that existing code
> to use sbitmaps as well.  I think doing that would be better for
> long-term maintenance perspectives, but I'm not going to insist that you
> do it now.

There's also the assert later but I've found a way to improve it slightly. While 
switching to auto_sbitmap I also changed the code slightly to allocate directly 
bitmaps to the right size. Since the change is probably bigger than what you had 
in mind I'd appreciate if you can give me an OK again. See updated patch in 
attachment. ChangeLog entry is unchanged:

2017-06-13  Thomas Preud'homme  <thomas.preudhomme@arm.com>

	* config/arm/arm.c (arm_option_override): Forbid ARMv8-M Security
	Extensions with more than 16 double VFP registers.
	(cmse_nonsecure_entry_clear_before_return): Remove second entry of
	to_clear_mask and all code related to it.  Replace the remaining
	entry by a sbitmap and adapt code accordingly.

> 
> As a result I'll let you take the call as to whether you keep this
> version or go back to your earlier patch.  If you do decide to keep this
> version, then see the comment below.

Given the changes I'm more happy with how the patch looks now and making it go 
in can be a nice incentive to change other ARMv8-M Security Extension related 
code later on.

Best regards,

Thomas