Bug 187991 - PaintingData' has no member named 'lightVector' on ARM_NEON
Summary: PaintingData' has no member named 'lightVector' on ARM_NEON
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: WebKit Local Build
Hardware: Other Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-07-25 01:51 PDT by Hongxu Jia
Modified: 2018-07-28 23:32 PDT (History)
3 users (show)

See Also:


Attachments
Patch (1.79 KB, patch)
2018-07-25 02:06 PDT, Hongxu Jia
no flags Details | Formatted Diff | Diff
Patch (1.97 KB, patch)
2018-07-25 02:12 PDT, Hongxu Jia
darin: review+
ews-watchlist: commit-queue-
Details | Formatted Diff | Diff
Archive of layout-test-results from ews107 for mac-sierra-wk2 (2.77 MB, application/zip)
2018-07-25 03:24 PDT, EWS Watchlist
no flags Details
Archive of layout-test-results from ews206 for win-future (12.92 MB, application/zip)
2018-07-25 13:32 PDT, EWS Watchlist
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Hongxu Jia 2018-07-25 01:51:11 PDT
In file included from /home/wrlbuild/builds/builds-9ba2c4db-26fe-412a-a205-c45e89dcf35d/wrlinux-std-sato_beaglebone-yocto_world_18/tmp-glibc/work/cortexa8hf-neon-wrs-linux-gnueabi/webkitgtk
/2.20.3-r0/webkitgtk-2.20.3/Source/WebCore/platform/graphics/filters/FELighting.cpp:32,
                 from DerivedSources/WebCore/unified-sources/UnifiedSource349.cpp:4:
/home/wrlbuild/builds/builds-9ba2c4db-26fe-412a-a205-c45e89dcf35d/wrlinux-std-sato_beaglebone-yocto_world_18/tmp-glibc/work/cortexa8hf-neon-wrs-linux-gnueabi/webkitgtk/2.20.3-r0/webkitgtk-2.20.3/Source/WebCore/platform/graphics/cpu/arm/filters/FELightingNEON.h: In member function 'void WebCore::FELighting::platformApplyNeon(c
onst WebCore::FELighting::LightingData&, const WebCore::LightSource::PaintingData&)':
/home/wrlbuild/builds/builds-9ba2c4db-26fe-412a-a205-c45e89dcf35d/wrlinux-std-sato_beaglebone-yocto_world_18/tmp-glibc/work/cortexa8hf-neon-wrs-linux-gnueabi/webkitgtk/2.20.3-r0/webkitgtk-2.20.3/Source/WebCore/platform/graphics/cpu/arm/filters/FELightingNEON.h:98:49: warning: requested alignment 16 is larger than 8 [-Wattributes]
     alignas(16) FELightingFloatArgumentsForNeon floatArguments;
                 
                                ^~~~~~~~~~~~~~
/home/wrlbuild/builds/builds-9ba2c4db-26fe-412a-a205-c45e89dcf35d/wrlinux-std-sato_beaglebone-yocto_world_18/tmp-glibc/work/cortexa8hf-neon-wrs-linux-gnueabi/webkitgtk/2.20.3-r0/webkitgtk-2.20.3/Source/WebCore/platform/graphics/cpu/arm/filters/FELightingNEON.h:147:46: error: 'const struct WebCore::LightSource::PaintingData' has no member named 'lightVector'; did you mean 'directionVector'?
         floatArguments.lightX = paintingData.lightVector.x(
);
                                              ^~~~~~~~~~~
                                              directionVector
/home/wrlbuild/builds/builds-9ba2c4db-26fe-412a-a205-c45e89dcf35d/wrlinux-std-sato_beaglebone-yocto_world_18/tmp-glibc/work/cortexa8hf-neon-wrs-linux-gnueabi/webkitgtk/2.20.3-r0/webkitgtk-2.20.3/Source/WebCore/platform/graphics/cpu/arm/filters/FELightingNEON.h:148:46: error: 'const struct WebCore::LightSource::PaintingData' has no member named 'lightVector'; did you mean 'di
rectionVector'?
         floatArguments.lightY = paintingData.lightVector.y();
                                              ^~~~~~~~~~~
                                              directionVector
/home/wrlbuild/builds/builds-9ba2c4db-26fe-412a-a205-c45e89dcf35d/wrlinux-std-sato_beaglebone-yocto_world_18/tmp-glibc/work/cortexa8hf-neon-wrs-linux-gnueabi/webkitgtk/2.20.3-r0/webkitgtk-2.20.3/Source/WebCore/platform/graphics/cpu/arm/filters/FELightingNEON.h:149:46: error: 'const struct WebCore::Li
ghtSource::PaintingData' has no member named 'lightVector'; did you mean 'directionVector'?
         floatArguments.lightZ = paintingData.lightVector.z();
                                              ^~~~~~~~~~~
                                              directionVector
Comment 1 Hongxu Jia 2018-07-25 01:54:40 PDT
    Since the following patch applied in upstream webkitgtk
    ...
    commit 1a55d8c685b3e5b4dbeda202009e7527aa59eadd
    Author: simon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
    Date:   Thu Nov 23 22:20:09 2017 +0000
    
        FELighting cleanup and optimization
        https://bugs.webkit.org/show_bug.cgi?id=179933
    ...
    
    It missed to tweak `paintingData.lightVector' which caused
    PaintingData' has no member named 'lightVector' on ARM_NEON
    (beaglebone)
Comment 2 Hongxu Jia 2018-07-25 02:06:41 PDT
Created attachment 345748 [details]
Patch
Comment 3 EWS Watchlist 2018-07-25 02:08:18 PDT
Attachment 345748 [details] did not pass style-queue:


ERROR: Source/WebCore/ChangeLog:8:  You should remove the 'No new tests' and either add and list tests, or explain why no new tests were possible.  [changelog/nonewtests] [5]
Total errors found: 1 in 2 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 4 Hongxu Jia 2018-07-25 02:12:41 PDT
Created attachment 345749 [details]
Patch
Comment 5 EWS Watchlist 2018-07-25 03:24:10 PDT
Comment on attachment 345749 [details]
Patch

Attachment 345749 [details] did not pass mac-wk2-ews (mac-wk2):
Output: https://webkit-queues.webkit.org/results/8648363

New failing tests:
fast/repaint/canvas-object-fit.html
Comment 6 EWS Watchlist 2018-07-25 03:24:12 PDT
Created attachment 345750 [details]
Archive of layout-test-results from ews107 for mac-sierra-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews107  Port: mac-sierra-wk2  Platform: Mac OS X 10.12.6
Comment 7 EWS Watchlist 2018-07-25 13:31:49 PDT
Comment on attachment 345749 [details]
Patch

Attachment 345749 [details] did not pass win-ews (win):
Output: https://webkit-queues.webkit.org/results/8652701

New failing tests:
http/tests/security/contentSecurityPolicy/userAgentShadowDOM/allow-video.html
Comment 8 EWS Watchlist 2018-07-25 13:32:00 PDT
Created attachment 345778 [details]
Archive of layout-test-results from ews206 for win-future

The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews206  Port: win-future  Platform: CYGWIN_NT-6.1-2.9.0-0.318-5-3-x86_64-64bit