[GitHub] [incubator-nuttx] pussuw commented on pull request #5917: arch/risc-v: move REGLOAD/REGSTORE macro to riscv_internal.h

[GitBox <gi...@apache.org>]

pussuw commented on pull request #5917:
URL: https://github.com/apache/incubator-nuttx/pull/5917#issuecomment-1084131476


   There is a build failure:
   
    In file included from common/riscv_pmp.c:34:
   common/riscv_pmp.c: In function 'pmp_read_region_cfg':
   Error: common/riscv_internal.h:155:20: error: declaration of 'tmp' shadows a previous local [-Werror=shadow]
         unsigned long tmp; \
                       ^~~
   common/riscv_pmp.c:74:21: note: in expansion of macro 'READ_CSR'
        uintptr_t tmp = READ_CSR(reg); \
                        ^~~~~~~~
   common/riscv_pmp.c:233:16: note: in expansion of macro 'PMP_READ_REGION_FROM_REG'
            return PMP_READ_REGION_FROM_REG(region, pmpcfg0);
                   ^~~~~~~~~~~~~~~~~~~~~~~~
   common/riscv_pmp.c:74:15: note: shadowed declaration is here
        uintptr_t tmp = READ_CSR(reg); \
                  ^~~
   common/riscv_pmp.c:233:16: note: in expansion of macro 'PMP_READ_REGION_FROM_REG'
            return PMP_READ_REGION_FROM_REG(region, pmpcfg0);
   
   Please, address this failure prior to merge.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org