<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>244508</bug_id>
          
          <creation_ts>2022-08-29 13:17:18 -0700</creation_ts>
          <short_desc>New GCC warnings after enabling USE_ANGLE_WEBGL</short_desc>
          <delta_ts>2022-09-05 13:18:18 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>ANGLE</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>NEW</bug_status>
          <resolution></resolution>
          
          <see_also>https://bugs.webkit.org/show_bug.cgi?id=244485</see_also>
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Minor</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Michael Catanzaro">mcatanzaro</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>dino</cc>
    
    <cc>kbr</cc>
    
    <cc>kkinnunen</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1894615</commentid>
    <comment_count>0</comment_count>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2022-08-29 13:17:18 -0700</bug_when>
    <thetext>I think bug #244485 introduced these GCC warnings:

[1447/6412] Building CXX object Source/ThirdParty/ANGLE/CM.../ANGLE.dir/src/libANGLE/renderer/gl/renderergl_utils.cpp.o
In file included from /home/mcatanzaro/Projects/WebKit/Source/ThirdParty/ANGLE/include/platform/FeaturesGL_autogen.h:13,
                 from /home/mcatanzaro/Projects/WebKit/Source/ThirdParty/ANGLE/src/libANGLE/renderer/gl/RendererGL.h:20,
                 from /home/mcatanzaro/Projects/WebKit/Source/ThirdParty/ANGLE/src/libANGLE/renderer/gl/ContextGL.h:14,
                 from /home/mcatanzaro/Projects/WebKit/Source/ThirdParty/ANGLE/src/libANGLE/renderer/gl/renderergl_utils.cpp:24:
/home/mcatanzaro/Projects/WebKit/Source/ThirdParty/ANGLE/src/libANGLE/renderer/gl/renderergl_utils.cpp: In function ‘void rx::nativegl_gl::InitializeFeatures(const rx::FunctionsGL*, angle::FeaturesGL*)’:
/home/mcatanzaro/Projects/WebKit/Source/ThirdParty/ANGLE/src/libANGLE/renderer/gl/renderergl_utils.cpp:2006:48: warning: suggest parentheses around ‘&amp;&amp;’ within ‘||’ [-Wparentheses]
 2006 |                             !isMesa || isIntel &amp;&amp; (Is9thGenIntel(device) || IsGeminiLake(device) ||
      |                                        ~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 2007 |                                                    IsCoffeeLake(device) || Is11thGenIntel(device) ||
      |                                                    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 2008 |                                                    Is12thGenIntel(device)));
      |                                                    ~~~~~~~~~~~~~~~~~~~~~~~
/home/mcatanzaro/Projects/WebKit/Source/ThirdParty/ANGLE/include/platform/Feature.h:19:36: note: in definition of macro ‘ANGLE_FEATURE_CONDITION’
   19 |         (set)-&gt;feature.enabled   = cond;                  \
      |                                    ^~~~
[1467/6412] Building CXX object Source/ThirdParty/ANGLE/CMakeFiles/ANGLE.dir/src/libANGLE/GLES1Renderer.cpp.o
/home/mcatanzaro/Projects/WebKit/Source/ThirdParty/ANGLE/src/libANGLE/GLES1Renderer.cpp: In copy constructor ‘gl::GLES1ShaderState::GLES1ShaderState(const gl::GLES1ShaderState&amp;)’:
/home/mcatanzaro/Projects/WebKit/Source/ThirdParty/ANGLE/src/libANGLE/GLES1Renderer.cpp:36:11: warning: ‘void* memcpy(void*, const void*, size_t)’ writing to an object of type ‘struct gl::GLES1ShaderState’ with no trivial copy-assignment; use copy-assignment or copy-initialization instead [-Wclass-memaccess]
   36 |     memcpy(this, &amp;other, sizeof(GLES1ShaderState));
      |     ~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from /home/mcatanzaro/Projects/WebKit/Source/ThirdParty/ANGLE/src/libANGLE/GLES1Renderer.cpp:9:
/home/mcatanzaro/Projects/WebKit/Source/ThirdParty/ANGLE/src/libANGLE/GLES1Renderer.h:56:8: note: ‘struct gl::GLES1ShaderState’ declared here
   56 | struct GLES1ShaderState
      |        ^~~~~~~~~~~~~~~~
[1489/6412] Building CXX object Source/ThirdParty/ANGLE/CMakeFiles/ANGLE.dir/src/libANGLE/State.cpp.o
/home/mcatanzaro/Projects/WebKit/Source/ThirdParty/ANGLE/src/libANGLE/State.cpp:10: warning: ignoring ‘#pragma clang diagnostic’ [-Wunknown-pragmas]
   10 | #pragma clang diagnostic ignored &quot;-Wglobal-constructors&quot;
      | 
[1490/6412] Building CXX object Source/ThirdParty/ANGLE/CMakeFiles/ANGLE.dir/src/libANGLE/angletypes.cpp.o
/home/mcatanzaro/Projects/WebKit/Source/ThirdParty/ANGLE/src/libANGLE/angletypes.cpp: In constructor ‘gl::SamplerState::SamplerState()’:
/home/mcatanzaro/Projects/WebKit/Source/ThirdParty/ANGLE/src/libANGLE/angletypes.cpp:173:11: warning: ‘void* memset(void*, int, size_t)’ clearing an object of type ‘class gl::SamplerState’ with no trivial copy-assignment; use assignment or value-initialization instead [-Wclass-memaccess]
  173 |     memset(this, 0, sizeof(SamplerState));
      |     ~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from /home/mcatanzaro/Projects/WebKit/Source/ThirdParty/ANGLE/src/libANGLE/angletypes.cpp:9:
/home/mcatanzaro/Projects/WebKit/Source/ThirdParty/ANGLE/src/libANGLE/angletypes.h:323:7: note: ‘class gl::SamplerState’ declared here
  323 | class SamplerState final
      |       ^~~~~~~~~~~~</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1896379</commentid>
    <comment_count>1</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2022-09-05 13:18:18 -0700</bug_when>
    <thetext>&lt;rdar://problem/99575256&gt;</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>