https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65787
--- Comment #2 from Jakub Jelinek ---
The formatting looks weird, look how the case UNSPEC: is formatted -
{ goes below case PARALLEL:, two columns to the right, then another two columns
to the right the body of the scope, then } below the {.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65787
--- Comment #3 from Jakub Jelinek ---
Though, len is used just in one place, so perhaps even better just remove the
{}s and use
if (XVECLEN (op, 0) != 2)
return 0;
and drop len variable alltogether, it will be more readable that wa
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65780
--- Comment #35 from H.J. Lu ---
Created attachment 35341
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=35341&action=edit
The final patch with variable name change and updated comments
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65779
Alan Modra changed:
What|Removed |Added
CC||amodra at gmail dot com
--- Comment #2 from
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65779
--- Comment #3 from Alan Modra ---
Re comment #1, insn 36 is not removed, but moved later to bb3
(insn 363 364 41 3 (set (reg/v:SI 0 0 [orig:239 s2 ] [239])
(lshiftrt:SI (reg/v:SI 3 3 [orig:265 adler ] [265])
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65787
--- Comment #4 from Bill Schmidt ---
Sorry, yeah, too much work on projects with different style guidelines. I
agree with removing the braces as in comment 3. Thanks!
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65779
--- Comment #4 from Alan Modra ---
Created attachment 35342
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=35342&action=edit
prototype patch
This seems to cure the testcase. I haven't bootstrapped it, and it probably
isn't ideal since I h
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65790
Bug ID: 65790
Summary: compilation error std::index_sequnece
Product: gcc
Version: 4.9.2
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: c++
101 - 108 of 108 matches
Mail list logo
