[webkit-reviews] review granted: [Bug 230061] Unbreak GCC_OFFLINEASM_SOURCE_MAP when LTO is in use : [Attachment 438488] Patch

[bugzilla-daemon at webkit.org]

Michael Catanzaro <mcatanzaro at gnome.org> has granted Angelos Oikonomopoulos
<angelos at igalia.com>'s request for review:
Bug 230061: Unbreak GCC_OFFLINEASM_SOURCE_MAP when LTO is in use
https://bugs.webkit.org/show_bug.cgi?id=230061

Attachment 438488: Patch

https://bugs.webkit.org/attachment.cgi?id=438488&action=review




--- Comment #4 from Michael Catanzaro <mcatanzaro at gnome.org> ---
Comment on attachment 438488
  --> https://bugs.webkit.org/attachment.cgi?id=438488
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=438488&action=review

Tested this. It works. Thanks Angelos!

> Source/JavaScriptCore/CMakeLists.txt:1472
> +	   COMPILE_OPTIONS "-fno-lto")

OK, I was going to whine that you need to check if the option is supported, but
this is within a CMAKE_COMPILER_IS_GNUCXX guard so no need.