[Openmp-commits] [PATCH] D65285: [OpenMP] Rename last file to cpp and remove LIBOMP_CFLAGS

Jonas Hahnfeld via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Thu Jul 25 08:55:13 PDT 2019


Hahnfeld created this revision.
Hahnfeld added reviewers: jlpeyton, tlwilmar, AndreyChurbanov.
Herald added subscribers: openmp-commits, guansong, mgorny.
Herald added a reviewer: jdoerfert.
Herald added a project: OpenMP.

All other files are already C++ and the build system has always
passed '-x c++' for C files, effectively compiling them as C++.

To stay warning free we need one fix in ittnotify_static.{c,cpp}:
The variable dll_path can be written to, so it must not be const.
GCC complained with -Wcast-qual and I think it's right.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D65285

Files:
  openmp/README.rst
  openmp/runtime/CMakeLists.txt
  openmp/runtime/cmake/LibompHandleFlags.cmake
  openmp/runtime/cmake/config-ix.cmake
  openmp/runtime/src/CMakeLists.txt
  openmp/runtime/src/thirdparty/ittnotify/ittnotify_static.c
  openmp/runtime/src/thirdparty/ittnotify/ittnotify_static.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D65285.211767.patch
Type: text/x-patch
Size: 7622 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20190725/a414e237/attachment.bin>


More information about the Openmp-commits mailing list