[webkit-changes] [WebKit/WebKit] e739d6: Simplify compiler flags checking in Source/ThirdPa...

Adrian Perez noreply at github.com
Tue Feb 27 23:56:53 PST 2024


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: e739d6576a590c7211f9438c5fa9d54fd4cb1ee6
      https://github.com/WebKit/WebKit/commit/e739d6576a590c7211f9438c5fa9d54fd4cb1ee6
  Author: Adrian Perez de Castro <aperez at igalia.com>
  Date:   2024-02-27 (Tue, 27 Feb 2024)

  Changed paths:
    M Source/ThirdParty/skia/CMakeLists.txt
    M Source/cmake/WebKitCompilerFlags.cmake

  Log Message:
  -----------
  Simplify compiler flags checking in Source/ThirdParty/skia/CMakeLists.txt
https://bugs.webkit.org/show_bug.cgi?id=270162

Reviewed by Michael Catanzaro.

Add new WEBKIT_CHECK_COMPILER_FLAGS and WEBKIT_ADD_COMPILER_FLAGS
functions, which split the existing functionality of the existing
WEBKIT_ADD_TARGET_{C,CXX}_FLAGS macros, making them more generic.
The old macros are kept for now, but implemented in terms of the
new functions.

While at it, the WEBKIT_ADD_COMPILER_FLAGS function was augmented to
support applying flags to SOURCE files and DIRECTORY paths, and a
tweak added to ensure results of checks for flags containing equals
characters (=) are properly cached by CMake to save some time in CMake
re-runs.

* Source/ThirdParty/skia/CMakeLists.txt: Make use of the new
  WEBKIT_CHECK_COMPILER_FLAGS and WEBKIT_ADD_COMPILER_FLAGS functions.
* Source/cmake/WebKitCompilerFlags.cmake: Split and improve
  functionality from the WEBKIT_ADD_TARGET_{C,CXX}_FLAGS macros into
  a new function to check flags and another generic function to apply
  flags to targets, sources, or directories.

Canonical link: https://commits.webkit.org/275430@main



To unsubscribe from these emails, change your notification settings at https://github.com/WebKit/WebKit/settings/notifications


More information about the webkit-changes mailing list