dim added inline comments.

INLINE COMMENTS
  Makefile.inc1:416 These are also unconditionally used in the previous block 
for WMAKENV, e.g.:
  
  ```
  WMAKEENV+=      CC="${XCC} ${XCFLAGS}" CXX="${XCXX} ${XCFLAGS} ${XCXXFLAGS}" \
                  DEPFLAGS="${DEPFLAGS}" \
                  CPP="${XCPP} ${XCFLAGS}" \
                  AS="${XAS}" AR="${XAR}" LD="${XLD}" NM=${XNM} \
                  OBJDUMP=${XOBJDUMP} OBJCOPY="${XOBJCOPY}" \
                  RANLIB=${XRANLIB} STRINGS=${XSTRINGS} \
                  SIZE="${XSIZE}"
  ```
  
  and just before that, they are unconditionally defined:
  
  ```
  XCOMPILERS=     CC CXX CPP
  .for COMPILER in ${XCOMPILERS}
  .if defined(CROSS_COMPILER_PREFIX)
  X${COMPILER}?=  ${CROSS_COMPILER_PREFIX}${${COMPILER}}
  .else
  X${COMPILER}?=  ${${COMPILER}}
  .endif
  .endfor
  XBINUTILS=      AS AR LD NM OBJCOPY OBJDUMP RANLIB SIZE STRINGS
  .for BINUTIL in ${XBINUTILS}
  .if defined(CROSS_BINUTILS_PREFIX)
  X${BINUTIL}?=   ${CROSS_BINUTILS_PREFIX}${${BINUTIL}}
  .else
  X${BINUTIL}?=   ${${BINUTIL}}
  .endif
  .endfor
  ```
  
  So I think it is unnecessary to check them again.

REVISION DETAIL
  https://reviews.freebsd.org/D2187

To: dim, imp, bapt, rodrigc
Cc: imp, freebsd-toolchain
_______________________________________________
freebsd-toolchain@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-toolchain
To unsubscribe, send any mail to "freebsd-toolchain-unsubscr...@freebsd.org"

Reply via email to