xiaoxiang781216 commented on code in PR #7154: URL: https://github.com/apache/incubator-nuttx/pull/7154#discussion_r976676895
########## arch/arm/src/common/Toolchain.defs: ########## @@ -87,30 +87,34 @@ endif # NuttX buildroot under Linux or Cygwin -ifeq ($(CONFIG_ENDIAN_BIG),y) Review Comment: can we always build two toolchain variables CROSSDEVLE and ARCHCPUFLAGSLE CROSSDEVBE and ARCHCPUFLAGSBE and then try CROSSDEVBE if it's good set CROSSDEV to CROSSDEVBE and ARCHCPUFLAGS to ARCHCPUFLAGSBE otherwise set CROSSDEV to CROSSDEVLE and ARCHCPUFLAGS to ARCHCPUFLAGSLE This process is more simpler I think. -- 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-unsubscr...@nuttx.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
