On 12.02.2016 18:16, Edgar E. Iglesias wrote: > On Fri, Feb 12, 2016 at 03:15:22PM +0000, Peter Maydell wrote: >> On 12 February 2016 at 15:12, Edgar E. Iglesias >> <edgar.igles...@gmail.com> wrote: >>> On Thu, Feb 11, 2016 at 07:11:48PM +0000, Peter Maydell wrote: >>>> The user-mode versions of get/set_r13_banked() exist just to assert >>>> if they're ever called -- the translate time code should never >>>> emit calls to them because SRS from user mode always UNDEF. >>>> There's no code in the softmmu versions that can't compile in >>>> CONFIG_USER_ONLY, so combine the two functions rather than >>>> having completely split versions under ifdefs. >>>> >>>> Signed-off-by: Peter Maydell <peter.mayd...@linaro.org> >>> Hi Peter, >>> >>> Do we really need the assert? >>> If we keep it, can't we have it for both -user and -softmmu (avoiding the >>> ifdef)? >> I would be happy to entirely drop the assert, yes. > OK, thanks. > > With that change: > Reviewed-by: Edgar E. Iglesias <edgar.igles...@xilinx.com> >
Yes, I also think it would be okay to drop that assert. Anyway: Reviewed-by: Sergey Fedorov <serge.f...@gmail.com>