[all-commits] [llvm/llvm-project] 8b5af3: [OpenMP] Change check for OS to check for defined ...

Brad Smith via All-commits all-commits at lists.llvm.org
Mon Dec 11 06:54:39 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 8b5af3139c18516433bc77d65dea59df50e052e9
      https://github.com/llvm/llvm-project/commit/8b5af3139c18516433bc77d65dea59df50e052e9
  Author: Brad Smith <brad at comstyle.com>
  Date:   2023-12-11 (Mon, 11 Dec 2023)

  Changed paths:
    M openmp/runtime/src/z_Linux_util.cpp

  Log Message:
  -----------
  [OpenMP] Change check for OS to check for defined for a macro (#75012)

Check for the existence of the macro instead of checking for Solaris.
illumos has this macro in sys/time.h.

/export/home/brad/llvm-brad/openmp/runtime/src/z_Linux_util.cpp:77:9: warning: 'TIMEVAL_TO_TIMESPEC' macro redefined [-Wmacro-redefined]
   77 | #define TIMEVAL_TO_TIMESPEC(tv, ts)                                            \
      |         ^
/usr/include/sys/time.h:424:9: note: previous definition is here
  424 | #define TIMEVAL_TO_TIMESPEC(tv, ts) { \
      |         ^




More information about the All-commits mailing list