Hi Richard, >> Yes, with a more general search loop we can get that case too - >> it doesn't trigger much though. The code that checks for this is >> now refactored into a new function. Given there are now many >> more calls to aarch64_bitmask_imm, I added a streamlined internal >> entry point that assumes the input is 64-bit. > > Sounds good, but could you put it before the mode version, > to avoid the forward declaration?
I can swap them around but the forward declaration is still required as aarch64_check_bitmask is 5000 lines before aarch64_bitmask_imm. Cheers, Wilco