https://gcc.gnu.org/bugzilla/show_bug.cgi?id=111366
Kewen Lin changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=111366
--- Comment #21 from CVS Commits ---
The releases/gcc-12 branch has been updated by Kewen Lin :
https://gcc.gnu.org/g:2b50ddce27be4738b494f5c4dd7083099b3de68c
commit r12-9925-g2b50ddce27be4738b494f5c4dd7083099b3de68c
Author: Kewen Lin
Date:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=111366
--- Comment #20 from CVS Commits ---
The releases/gcc-13 branch has been updated by Kewen Lin :
https://gcc.gnu.org/g:4d22de945430e6f8c33ef98c90a4fad8de0ef41e
commit r13-7953-g4d22de945430e6f8c33ef98c90a4fad8de0ef41e
Author: Kewen Lin
Date:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=111366
--- Comment #19 from CVS Commits ---
The master branch has been updated by Kewen Lin :
https://gcc.gnu.org/g:a65b38e361320e0aa45adbc969c704385ab1f45b
commit r14-4247-ga65b38e361320e0aa45adbc969c704385ab1f45b
Author: Kewen Lin
Date: Mon Sep
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=111366
--- Comment #18 from Jan Wassenberg ---
Ah, got it. We'll change the pragma to ",htm" as you suggest. Thank you :)
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=111366
--- Comment #17 from Kewen Lin ---
(In reply to Mathieu Malaterre from comment #16)
> Interesting, the following works for me:
>
> % /usr/bin/c++ -O1 -mcpu=power8 -mno-htm -flto=auto -c skeleton_test.cc
Yeah, the suggestion on an extra
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=111366
--- Comment #16 from Mathieu Malaterre ---
Interesting, the following works for me:
% /usr/bin/c++ -O1 -mcpu=power8 -mno-htm -flto=auto -c skeleton_test.cc
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=111366
--- Comment #15 from Mathieu Malaterre ---
For some reason the no-htm flag does not seems to work in my case:
% /usr/bin/c++ -O1 -mcpu=power8 -flto=auto -c skeleton_test.cc
skeleton_test.cc: In member function 'TestFloorLog2::operator()
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=111366
Kewen Lin changed:
What|Removed |Added
Assignee|unassigned at gcc dot gnu.org |linkw at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=111366
Kewen Lin changed:
What|Removed |Added
Status|REOPENED|NEW
--- Comment #13 from Kewen Lin ---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=111366
Kewen Lin changed:
What|Removed |Added
Ever confirmed|0 |1
Resolution|INVALID
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=111366
--- Comment #11 from Mathieu Malaterre ---
(In reply to Kewen Lin from comment #9)
> It looks to me that the error message is expected, because the source code
> forces the function as always_inline, users would like it to be inlined
> always,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=111366
--- Comment #10 from Kewen Lin ---
(In reply to Jan Wassenberg from comment #5)
> Thanks for reporting this. One might think this is caused by -mcpu=power9
> clashing with our #pragma target altivec,vsx,power8-vector.
>
> However, what makes
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=111366
Kewen Lin changed:
What|Removed |Added
Resolution|--- |INVALID
Status|UNCONFIRMED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=111366
--- Comment #8 from Mathieu Malaterre ---
Further reduced:
#include
#define HWY_PRAGMA(tokens) _Pragma(#tokens)
#define HWY_PUSH_ATTRIBUTES(targets_str) HWY_PRAGMA(GCC target targets_str)
__attribute__((always_inline)) void PreventElision() {
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=111366
--- Comment #7 from Mathieu Malaterre ---
In case this helps resolve this, here is the cvise-reduced code:
% cat skeleton_test.cc
#include
#define HWY_PRAGMA(tokens) _Pragma(#tokens)
namespace hwy {
#define HWY_PUSH_ATTRIBUTES(targets_str)
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=111366
--- Comment #6 from Jan Wassenberg ---
Thinking about this more, the LTO means more opportunity for inlining and thus
for the compiler to hit the legit "don't want to inline POWER9 into POWER8"
error.
Interestingly this does not happen on x86
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=111366
Jan Wassenberg changed:
What|Removed |Added
CC||jan.wassenberg at gmail dot com
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=111366
--- Comment #4 from Mathieu Malaterre ---
reduction started:
% cvise check.sh skeleton_test.cc
00:00:02 INFO ===< 4165591 >===
00:00:02 INFO running 4 interestingness tests in parallel
00:00:02 INFO INITIAL PASSES
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=111366
--- Comment #3 from Mathieu Malaterre ---
compilation flags required are: `-O1 -mcpu=power9 -flto=auto`
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=111366
--- Comment #2 from Mathieu Malaterre ---
Typical setup:
* https://gcc.godbolt.org/z/5W45a6j8x
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=111366
--- Comment #1 from Mathieu Malaterre ---
Full log:
*
https://launchpadlibrarian.net/684588621/buildlog_ubuntu-mantic-ppc64el.highway_1.0.7-1_BUILDING.txt.gz
22 matches
Mail list logo
