[Bug tree-optimization/55995] New: vect increase_alignment notes missing from dump file

[Bug tree-optimization/55995] New: vect increase_alignment notes missing from dump file

[janis at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55995

             Bug #: 55995
           Summary: vect increase_alignment notes missing from dump file
    Classification: Unclassified
           Product: gcc
           Version: 4.7.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: tree-optimization
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: janis@gcc.gnu.org
                CC: singhai@gcc.gnu.org
            Target: arm*-*-* powerpc*-*-*


Test gcc.dg/vect/aligned-section-anchors-nest-1.c started failing with the
patch added in revisions 191883 and 191884 that changes the verbosity of
compiler dump files.  The dump file scan is for a file created with
-fdump-ipa-increase-alignment and a message output with MSG_NOTE.  Unlike
-fdump-tree-switch-options, there is apparently no support for
-fdump-ipa-switch-options and I couldn't figure out how to get the notes
included in the increase_alignment dump file.

The messages are output from function increase_alignment in tree-vectorizer.c.

The test is run for effective targets section_anchors and vect_int, which
includes powerpc*-*-* and arm*-*-*.

The test was added in 2009 so this is a regression.

[Bug tree-optimization/55995] vect increase_alignment notes missing from dump file

[rguenth at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55995

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |WAITING
   Last reconfirmed|                            |2013-01-16
     Ever Confirmed|0                           |1

--- Comment #1 from Richard Biener <rguenth at gcc dot gnu.org> 2013-01-16 10:09:25 UTC ---
Index: gcc/testsuite/gcc.dg/vect/vect.exp
===================================================================
--- gcc/testsuite/gcc.dg/vect/vect.exp  (revision 195232)
+++ gcc/testsuite/gcc.dg/vect/vect.exp  (working copy)
@@ -156,7 +156,7 @@ dg-runtest [lsort [glob -nocomplain $src

 # alignment-sensitive -fsection-anchors tests
 set DEFAULT_VECTCFLAGS $SAVED_DEFAULT_VECTCFLAGS
-lappend DEFAULT_VECTCFLAGS "-fsection-anchors" "-fdump-ipa-increase_alignment"
+lappend DEFAULT_VECTCFLAGS "-fsection-anchors"
"-fdump-ipa-increase_alignment-details"
 dg-runtest [lsort [glob -nocomplain
$srcdir/$subdir/aligned-section-anchors-*.\[cS\]]]  \
        "" $DEFAULT_VECTCFLAGS


should work, can you verify that?

[Bug tree-optimization/55995] vect increase_alignment notes missing from dump file

[janis at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55995

--- Comment #2 from Janis Johnson <janis at gcc dot gnu.org> 2013-01-16 16:38:24 UTC ---
Interesting, it causes the compiler to segfault on both arm-none-eabi and
powerpc-none-eabi:

/scratch/janisjo/build6/fsf-arm-eabi/src/gcc-mainline/gcc/testsuite/gcc.dg/vect/aligned-section-anchors-nest-1.c:31:1:
internal compiler error: Segmentation fault^M
0x8605072 crash_signal^M
       
/scratch/janisjo/build6/fsf-arm-eabi/src/gcc-mainline/gcc/toplev.c:332^M
0x82f5523 contains_struct_check^M
        /scratch/janisjo/build6/fsf-arm-eabi/src/gcc-mainline/gcc/tree.h:3782^M
0x82f5523 dump_loc^M
       
/scratch/janisjo/build6/fsf-arm-eabi/src/gcc-mainline/gcc/dumpfile.c:266^M
0x82f56a2 dump_printf_loc(int, unsigned int, char const*, ...)^M
       
/scratch/janisjo/build6/fsf-arm-eabi/src/gcc-mainline/gcc/dumpfile.c:370^M
0x882ffc7 increase_alignment^M
       
/scratch/janisjo/build6/fsf-arm-eabi/src/gcc-mainline/gcc/tree-vectorizer.c:238^M
Please submit a full bug report,^M
with preprocessed source if appropriate.^M
Please include the complete backtrace with any bug report.^M
See <http://gcc.gnu.org/bugs.html> for instructions.^M

FAIL: gcc.dg/vect/aligned-section-anchors-nest-1.c (internal compiler error)
FAIL: gcc.dg/vect/aligned-section-anchors-nest-1.c (test for excess errors)

[Bug tree-optimization/55995] vect increase_alignment notes missing from dump file

[singhai at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55995

Sharad Singhai <singhai at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|WAITING                     |ASSIGNED
         AssignedTo|unassigned at gcc dot       |singhai at gcc dot gnu.org
                   |gnu.org                     |

--- Comment #3 from Sharad Singhai <singhai at gcc dot gnu.org> 2013-01-16 17:03:26 UTC ---
Hmm, it looks like it is trying to output a source location where none exists.
This is clearly a bug. I would look at it.

Thanks,
Sharad

[Bug tree-optimization/55995] vect increase_alignment notes missing from dump file

[singhai at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55995

--- Comment #4 from Sharad Singhai <singhai at gcc dot gnu.org> 2013-01-18 18:26:17 UTC ---
Author: singhai
Date: Fri Jan 18 18:26:04 2013
New Revision: 195303

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=195303
Log:
2013-01-18  Sharad Singhai  <singhai@google.com>

    PR tree-optimization/55995
    * dumpfile.c (dump_loc): Print location only if available.
    * tree-vectorizer.c (increase_alignment): Intialize vect_location.

testsuite/ChangeLog
    PR tree-optimization/55995
    * gcc.dg/vect/vect.exp: Use "details" flags for dump info.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/dumpfile.c
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/gcc.dg/vect/vect.exp
    trunk/gcc/tree-vectorizer.c

[Bug tree-optimization/55995] vect increase_alignment notes missing from dump file

[singhai at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55995

Sharad Singhai <singhai at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED

--- Comment #5 from Sharad Singhai <singhai at gcc dot gnu.org> 2013-01-19 01:24:43 UTC ---
Should be fixed in r195303.