[Bug tree-optimization/78930] Chromium failed compilation with -finline-functions or -O3

[trippels at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78930

Markus Trippelsdorf  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 CC||trippels at gcc dot gnu.org
 Resolution|--- |INVALID

--- Comment #1 from Markus Trippelsdorf  ---
Not a gcc bug. 
You can use the patch in
https://bugs.chromium.org/p/chromium/issues/detail?id=593980 or version 57,
where the issue was fixed finally.

[Bug tree-optimization/78930] New: Chromium failed compilation with -finline-functions or -O3

[mad_sam at bk dot ru]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78930

Bug ID: 78930
   Summary: Chromium failed compilation with -finline-functions or
-O3
   Product: gcc
   Version: 5.4.0
Status: UNCONFIRMED
  Severity: normal
  Priority: P3
 Component: tree-optimization
  Assignee: unassigned at gcc dot gnu.org
  Reporter: mad_sam at bk dot ru
  Target Milestone: ---

Created attachment 40412
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=40412=edit
environment

Part of log:

[23851/23852] touch obj/chrome/browser_dependencies.stamp
[23852/23852] x86_64-pc-linux-gnu-g++ -pie -fPIC -Wl,-z,noexecstack -Wl,-z,now
-Wl,-z,relro -Wl,-z,defs -Wl,--no-as-needed -lpthread -Wl,--as-needed -pthread
-m64 -Wl,-O1 -Wl,--gc-sections -Wl,-rpath-link=. -Wl,--disable-new-dtags
-Wl,--export-dynamic -Wl,-O1 -Wl,--as-needed -o "./chrome" -Wl,--start-group
@"./chrome.rsp"  -Wl,--end-group  -ldl -lrt -lpangocairo-1.0 -lpango-1.0
-lgobject-2.0 -lglib-2.0 -lcairo -lX11 -lX11-xcb -lxcb -lXcomposite -lXcursor
-lXdamage -lXext -lXfixes -lXi -lXrender -lXtst -lgmodule-2.0 -lgthread-2.0
-licui18n -licuuc -licudata -lsmime3 -lnss3 -lnssutil3 -lplds4 -lplc4 -lnspr4
-lexpat -lxml2 -lfontconfig -ldbus-1 -latomic -lgconf-2 -lresolv -lgio-2.0 -lz
-lharfbuzz-icu -lharfbuzz -lpng16 -lfreetype -ljpeg -lXss -lre2 -lXrandr
-lavcodec -lavformat -lavutil -lvpx -lm -lasound -latk-1.0 -lminizip -lsnappy
-lFLAC -lgtk-3 -lgdk-3 -lcairo-gobject -lgdk_pixbuf-2.0 -lwebp -lwebpdemux
-lxslt
FAILED: chrome
x86_64-pc-linux-gnu-g++ -pie -fPIC -Wl,-z,noexecstack -Wl,-z,now -Wl,-z,relro
-Wl,-z,defs -Wl,--no-as-needed -lpthread -Wl,--as-needed -pthread -m64 -Wl,-O1
-Wl,--gc-sections -Wl,-rpath-link=. -Wl,--disable-new-dtags
-Wl,--export-dynamic -Wl,-O1 -Wl,--as-needed -o "./chrome" -Wl,--start-group
@"./chrome.rsp"  -Wl,--end-group  -ldl -lrt -lpangocairo-1.0 -lpango-1.0
-lgobject-2.0 -lglib-2.0 -lcairo -lX11 -lX11-xcb -lxcb -lXcomposite -lXcursor
-lXdamage -lXext -lXfixes -lXi -lXrender -lXtst -lgmodule-2.0 -lgthread-2.0
-licui18n -licuuc -licudata -lsmime3 -lnss3 -lnssutil3 -lplds4 -lplc4 -lnspr4
-lexpat -lxml2 -lfontconfig -ldbus-1 -latomic -lgconf-2 -lresolv -lgio-2.0 -lz
-lharfbuzz-icu -lharfbuzz -lpng16 -lfreetype -ljpeg -lXss -lre2 -lXrandr
-lavcodec -lavformat -lavutil -lvpx -lm -lasound -latk-1.0 -lminizip -lsnappy
-lFLAC -lgtk-3 -lgdk-3 -lcairo-gobject -lgdk_pixbuf-2.0 -lwebp -lwebpdemux
-lxslt
obj/third_party/WebKit/Source/core/css/libcss_4.a(CSSToStyleMap.o): In function
`blink::CSSToStyleMap::mapFillXPosition(blink::StyleResolverState&,
blink::FillLayer*, blink::CSSValue const&)':
CSSToStyleMap.cpp:(.text._ZN5blink13CSSToStyleMap16mapFillXPositionERNS_18StyleResolverStateEPNS_9FillLayerERKNS_8CSSValueE+0x65):
undefined reference to `blink::Length
blink::StyleBuilderConverter::convertPositionLength<(blink::CSSValueID)169,
(blink::CSSValueID)170>(blink::StyleResolverState&, blink::CSSValue const&)'
obj/third_party/WebKit/Source/core/css/libcss_4.a(CSSToStyleMap.o): In function
`blink::CSSToStyleMap::mapFillYPosition(blink::StyleResolverState&,
blink::FillLayer*, blink::CSSValue const&)':
CSSToStyleMap.cpp:(.text._ZN5blink13CSSToStyleMap16mapFillYPositionERNS_18StyleResolverStateEPNS_9FillLayerERKNS_8CSSValueE+0x65):
undefined reference to `blink::Length
blink::StyleBuilderConverter::convertPositionLength<(blink::CSSValueID)165,
(blink::CSSValueID)166>(blink::StyleResolverState&, blink::CSSValue const&)'
collect2: error: ld returned 1 exit status
ninja: build stopped: subcommand failed.
 * ERROR: www-client/chromium-56.0.2924.21::gentoo failed (compile phase):
 *   (no error message)
 *
 * Call stack:
 * ebuild.sh, line 115:  Called src_compile
 *   environment, line 5224:  Called die
 * The specific snippet of code:
 *   eninja -C out/Release ${ninja_targets} || die;
 *
 * If you need support, post the output of `emerge --info
'=www-client/chromium-56.0.2924.21::gentoo'`,
 * the complete build log and the output of `emerge -pqv
'=www-client/chromium-56.0.2924.21::gentoo'`.
 *
 * You have enabled the custom-cflags USE flag.
 * Please disable it before reporting a bug.
 *
 *
 * MemTotal:7610284 kB
 * SwapTotal: 0 kB

Full build log (55mb): https://yadi.sk/d/GdI8O6ls34c5Qt

[Bug target/78904] zero-extracts are not effective

[ubizjak at gmail dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78904

Uroš Bizjak  changed:

   What|Removed |Added

   Assignee|unassigned at gcc dot gnu.org  |ubizjak at gmail dot com

--- Comment #10 from Uroš Bizjak  ---
Fixed.

[Bug target/78904] zero-extracts are not effective

[ubizjak at gmail dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78904

Uroš Bizjak  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
  Component|middle-end  |target
 Resolution|--- |FIXED
   Target Milestone|--- |7.0

--- Comment #9 from Uroš Bizjak  ---
Target issue, fixed by rewriting relevant patterns to the canonical form.

[Bug middle-end/78904] zero-extracts are not effective

[uros at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78904

--- Comment #8 from uros at gcc dot gnu.org ---
Author: uros
Date: Mon Dec 26 19:00:47 2016
New Revision: 243929

URL: https://gcc.gnu.org/viewcvs?rev=243929=gcc=rev
Log:
PR target/78904
* config/i386/i386.md (addqi_ext_1): Canonicalize insn pattern w.r.t.
zero_extract RTXes.
(*addqi_ext_2): Ditto.
(testqi_ext_ccno_0): Canonicalize expander w.r.t. zero_extract RTXes.
(testqi_ext_1_ccno): Rename from testqi_ext_ccno_0.
(*testqi_ext_0): Merge with *testqi_ext_1.
(*testqi_ext_1): Canonicalize insn pattern w.r.t. zero_extract RTXes.
Update corresponding splitter.
(*testqi_ext_2): Canonicalize insn pattern w.r.t. zero_extract RTXes.
(*andqi_ext_0): Merge with *andqi_ext_1.
(andqi_ext_1): Canonicalize insn pattern w.r.t. zero_extract RTXes.
Rename from *andqi_ext_1.  Update corresponding splitter and
peephole2 patterns.
(*andqi_ext_1_cc): Rename from *andqi_ext_0_cc.
(*andqi_ext_2): Canonicalize insn pattern w.r.t. zero_extract RTXes.
(*qi_ext_0): Merge with *andqi_ext_1.
(*qi_ext_1): Canonicalize insn pattern w.r.t.
zero_extract RTXes.  Update corresponding splitter.
(*qi_ext_2): Canonicalize insn pattern w.r.t.
zero_extract RTXes.
(xorqi_cc_ext_1): Canonicalize expander w.r.t. zero_extract RTXes.
(xorqi_ext_1_cc): Rename from xorqi_cc_ext_1.
(*xorqi_cc_ext_1): Canonicalize insn pattern w.r.t. zero_extract RTXes.
Update corresponding splitter.
(*xorqi_ext_1_cc): Rename from *xorqi_cc_ext_1.
(isinfxf2): Update calls to renamed expanders.
(isinf2): Ditto.
* config/i386/i386.c (ix86_expand_fp_compare): Ditto.
(ix86_emit_fp_unordered_jump): Ditto.
(ix86_emit_i387_round): Ditto.

testsuite/ChangeLog:

PR target/78904
* gcc.target/i386/pr78904.c: New test.


Added:
trunk/gcc/testsuite/gcc.target/i386/pr78904.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/config/i386/i386.c
trunk/gcc/config/i386/i386.md
trunk/gcc/testsuite/ChangeLog

[Bug other/67520] libmpx should abort() instead of exit(255) on bound violation detection

[aivchenk at gmail dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67520

Alexander Ivchenko  changed:

   What|Removed |Added

 CC||aivchenk at gmail dot com

--- Comment #2 from Alexander Ivchenko  ---
(In reply to Richard Biener from comment #1)
> IMHO libraries should have a way to specify a custom handler (and default
> that to abort maybe).  Indeed exit (255) isn't very canonical.

Addressed in:


r243928 | aivchenk | 2016-12-26 18:14:06 +0300 (Mon, 26 Dec 2016) | 14 lines

2016-12-26  Alexander Ivchenko  

* mpxrt/libtool-version: New version.
* mpxrt/mpxrt-utils.c (set_mpx_rt_stop_handler): New function.
(print_help): Add help for CHKP_RT_STOP_HANDLER environment
variable.
(__mpxrt_init_env_vars): Add initialization of stop_handler.
(__mpxrt_stop_handler): New function.
(__mpxrt_stop): Ditto.
* mpxrt/mpxrt-utils.h (mpx_rt_stop_mode_handler_t): New enum.
* mpxrt/mpxrt.c (handler): Replace exit(255) with __mpxrt_stop
function call.

Now libmpx will call abort on bounds violations by default (User can restore
previous behavior with  CHKP_RT_STOP_HANDLER env variable)

[Bug target/78860] ICE in compilation for MIPS with -mpaired-single

[niva at niisi dot msk.ru]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78860

niva at niisi dot msk.ru changed:

   What|Removed |Added

  Known to fail||6.3.0

--- Comment #2 from niva at niisi dot msk.ru ---
The error was reproduced also with mips64-none-elf-gcc-6.3.0

[Bug middle-end/63994] Ada bootstrap fails with -fcheck-pointer-bounds -mmpx

[aivchenk at gmail dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63994

Alexander Ivchenko  changed:

   What|Removed |Added

 CC||aivchenk at gmail dot com

--- Comment #8 from Alexander Ivchenko  ---
I couldn't reproduce the issue as of Dec 26 trunk. I suppose it can be closed

[Bug fortran/52473] CSHIFT slow - inline it?

[tkoenig at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=52473

--- Comment #4 from Thomas Koenig  ---
Looking at the code, inlining cshift for a constant shift
could already be a good idea.

So, change

b = cshift(b,1) + cshift(b,-1)

to

a = b

b(1) = a(2) + a(n)
b(2:n-1) = a(1:n-2) + a(3:n)
b(n) = a(1) + a(n-1)

[Bug c++/78929] Incorrect implementation of Refining Expression Evaluation Order for Idiomatic C++

[trippels at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78929

Markus Trippelsdorf  changed:

   What|Removed |Added

 Resolution|FIXED   |INVALID

[Bug c++/78929] Incorrect implementation of Refining Expression Evaluation Order for Idiomatic C++

[trippels at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78929

Markus Trippelsdorf  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 CC||trippels at gcc dot gnu.org
 Resolution|--- |FIXED

--- Comment #3 from Markus Trippelsdorf  ---
Closing as invalid.

[Bug c++/78929] Incorrect implementation of Refining Expression Evaluation Order for Idiomatic C++

[boostcpp at gmail dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78929

--- Comment #2 from Ryou Ezoe  ---
My bad.
§1.10 p18 still said "i++ + i" is undefined.

[Bug libstdc++/60936] [5/6/7 Regression] Binary code bloat with std::string

[d.v.a at ngs dot ru]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60936

--- Comment #23 from __vic  ---
Jonathan, have you tried to merge you patch with mine? Yours lacks decoupling
of string and iostream in c++11/cow-string-inst.o. I think it's a reason why
code size was unaffected