[Bug tree-optimization/38051] [4.4 Regression] Miscompilation of glibc's memcmp
--- Comment #15 from rguenth at gcc dot gnu dot org 2008-11-20 12:15 --- *** Bug 38169 has been marked as a duplicate of this bug. *** -- rguenth at gcc dot gnu dot org changed: What|Removed |Added CC||holger dot hopp at sap dot ||com http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38051
[Bug tree-optimization/38169] Wrong string constant optimizing
--- Comment #5 from rguenth at gcc dot gnu dot org 2008-11-20 12:15 --- Fixed. *** This bug has been marked as a duplicate of 38051 *** -- rguenth at gcc dot gnu dot org changed: What|Removed |Added Status|ASSIGNED|RESOLVED Keywords||alias, wrong-code Resolution||DUPLICATE Target Milestone|--- |4.4.0 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38169
[Bug tree-optimization/37868] [4.3 Regression] code that breaks TBAA is misoptimized even with -fno-strict-aliasing
--- Comment #9 from rguenth at gcc dot gnu dot org 2008-11-20 12:14 --- Fixed. -- rguenth at gcc dot gnu dot org changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution||FIXED http://gcc.gnu.org/bugzilla/show_bug.cgi?id=37868
[Bug tree-optimization/37868] [4.3 Regression] code that breaks TBAA is misoptimized even with -fno-strict-aliasing
--- Comment #8 from rguenth at gcc dot gnu dot org 2008-11-20 12:13 --- Subject: Bug 37868 Author: rguenth Date: Thu Nov 20 12:12:01 2008 New Revision: 142040 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=142040 Log: 2008-11-20 Richard Guenther <[EMAIL PROTECTED]> PR tree-optimization/37868 * tree-ssa-structalias.c (set_uids_in_ptset): Add SFTs based on pointed to variable and access size. * gcc.dg/torture/pr37868.c: New testcase. * gcc.c-torture/execute/pr38048-1.c: Likewise. * gcc.c-torture/execute/pr38048-2.c: Likewise. Backport from mainline: 2008-07-07 Richard Guenther <[EMAIL PROTECTED]> * tree-ssa-structalias.c (struct variable_info): Add is_full_var flag. (new_var_info): Set it to false. (solution_set_add): Correctly handle pointers outside a var and inside a field. (type_safe): Treat variables with is_full_var properly. (do_sd_constraint): Likewise. (do_ds_constraint): Likewise. (process_constraint): Remove zeroing offset for !use_field_sensitive. (get_constraint_for_ptr_offset): New function. (get_constraint_for_component_ref): Handle is_full_vars properly. (get_constraint_for): Handle POINTER_PLUS_EXPR. (handle_ptr_arith): Remove. (find_func_aliases): Handle POINTER_PLUS_EXPR through generic get_constraint_for code. (create_function_info_for): For parameter and result varinfos set is_full_var flag. (create_variable_info_for): Set is_full_var flag whenever we just created a single varinfo for a decl. (init_alias_vars): Initialize use_field_sensitive from max-fields-for-field-sensitive parameter. * gcc.dg/torture/pta-ptrarith-1.c: New testcase. * gcc.dg/torture/pta-ptrarith-2.c: Likewise. Added: branches/gcc-4_3-branch/gcc/testsuite/gcc.c-torture/execute/pr38048-1.c branches/gcc-4_3-branch/gcc/testsuite/gcc.c-torture/execute/pr38048-2.c branches/gcc-4_3-branch/gcc/testsuite/gcc.dg/torture/pr37868.c branches/gcc-4_3-branch/gcc/testsuite/gcc.dg/torture/pta-ptrarith-1.c branches/gcc-4_3-branch/gcc/testsuite/gcc.dg/torture/pta-ptrarith-2.c Modified: branches/gcc-4_3-branch/gcc/ChangeLog branches/gcc-4_3-branch/gcc/testsuite/ChangeLog branches/gcc-4_3-branch/gcc/tree-ssa-structalias.c -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=37868
[Bug fortran/38181] calls to SIZE not optimized out of loops
--- Comment #8 from jakub at gcc dot gnu dot org 2008-11-20 12:07 --- Fixed. -- jakub at gcc dot gnu dot org changed: What|Removed |Added Status|NEW |RESOLVED Resolution||FIXED http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38181
[Bug java/38198] New: DefaultTableColumnModel.java: redundant call to new
I just had a look at some of the source code of the Java package in the GNU gcc version 4.4.0 snapshot 20081114 gcc-4.4-20081114/libjava/classpath/javax/swing/table/DefaultTableColumnModel.java:401 Avoid unused local variables such as 'ls'. The source code is java.util.ArrayList ls = new java.util.ArrayList(); Suggest remove unused local variable ls. -- Summary: DefaultTableColumnModel.java: redundant call to new Product: gcc Version: 4.4.0 Status: UNCONFIRMED Severity: minor Priority: P3 Component: java AssignedTo: unassigned at gcc dot gnu dot org ReportedBy: dcb314 at hotmail dot com GCC host triplet: suse-linux-x86_64 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38198
[Bug java/38197] New: JobSheetsSupported.java: unused local variable
I just had a look at some of the source code of the Java package in
the GNU gcc version 4.4.0 snapshot 20081114
gcc-4.4-20081114/libjava/classpath/gnu/javax/print/ipp/attribute/supported/JobSheetsSupported.java:138
Avoid unused local variables such as 'j'.
The source code is
int j = 0;
while (it.hasNext())
{
tmp = (JobSheetsSupported) it.next();
Attribute att = tmp.getAssociatedAttribute();
if (att != null)
result.add(att);
j++;
}
Suggest remove unused local variable j.
--
Summary: JobSheetsSupported.java: unused local variable
Product: gcc
Version: 4.4.0
Status: UNCONFIRMED
Severity: minor
Priority: P3
Component: java
AssignedTo: unassigned at gcc dot gnu dot org
ReportedBy: dcb314 at hotmail dot com
GCC host triplet: suse-linux-x86_64
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38197
[Bug fortran/38181] calls to SIZE not optimized out of loops
--- Comment #7 from jv244 at cam dot ac dot uk 2008-11-20 11:03 --- (In reply to comment #6) great.. thanks. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38181
[Bug libstdc++/38196] num_put<>::do_put(bool) performs 'internal' padding incorrectly when boolalpha==true
--- Comment #1 from tsyvarev at ispras dot ru 2008-11-20 10:57 ---
Example:
#include
#include
using namespace std;
class my_numpunct : public numpunct
{
protected:
string do_falsename() const {return "-no-";}
};
int main()
{
locale my_loc = locale(locale::classic(), new my_numpunct());
cout.imbue(my_loc);
cout.width(6);
bool b = false;
cout << internal << boolalpha << b << endl;
return 0;
}
output "- no-" instead of " -no-"
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38196
[Bug libstdc++/38196] New: num_put<>::do_put(bool) performs 'internal' padding incorrectly when boolalpha==true
The description of num_put<>::do_put(bool) function states (22.2.2.2.2):
iter_type do_put(iter_type out, ios_base& str, char_type fill, bool val) const;
Effects: If (str.flags()&ios_base::boolalpha)==0 then do
out = do_put(out, str, fill, (int)val)
Otherwise do
const numpunct& np = use_facet >(loc);
string_type s = val ? np.truename() : np.falsename();
and then insert the characters of s into out.
It is not specified how exactly the insertion the characters of s into out
iterator is performed. It seems that the padding of the string is done the same
way as for the other functions from num_put<>::do_put() family, namely
(22.2.2.2.2 p19 Table 61):
adjustfield == iosbase::left - pad after
adjustfield == iosbase::right - pad before
adjustfield == internal and a sign occurs in the representation - pad after the
sign
adjustfield == internal and representation after stage 1 began with 0x or 0X -
pad after x or X
otherwise pad before
num_put<>::do_put(bool), however, should output a string that in general is not
a string representation of some number. It makes no sense for a string that can
be arbitrary to process the sign symbols, "0x" or "0X" in a different way than
its other substrings. That means, the 'internal' flag should be functionally
equivalent to 'right' ("pad before") in this case.
This is exactly the way the operator for inserting sequences of symbols into a
stream (operator<< for const charT*) is implemented. Although the description
of padding in this case (27.6.2.5.4, p2) refers to the description of
num_put<>::do_put, in practice the 'internal' flag is functionally equivalent
to 'right' in this case, which makes sense.
However, the implementation of num_put<>::do_put(bool) performs 'internal'
padding not like operator<< for sequences of characters but rather the same way
as the other functions from num_put<>::do_put family: it inserts fill symbols
between the 'sign' or "0x"/"0X" and the rest of the string.
--
Summary: num_put<>::do_put(bool) performs 'internal' padding
incorrectly when boolalpha==true
Product: gcc
Version: unknown
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: libstdc++
AssignedTo: unassigned at gcc dot gnu dot org
ReportedBy: tsyvarev at ispras dot ru
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38196
[Bug java/38195] New: ClassRmicCompiler.java: remove unused variable
I just had a look at some of the source code of the Java package in the GNU gcc version 4.4.0 snapshot 20081114 gcc-4.4-20081114/libjava/classpath/tools/gnu/classpath/tools/rmic/ClassRmicCompiler.java:922 Avoid unused local variables such as'endReturnTryCatch'. The source code is Label endReturnTryCatch = new Label(); Suggest remove unused local variable endReturnTryCatch. -- Summary: ClassRmicCompiler.java: remove unused variable Product: gcc Version: 4.4.0 Status: UNCONFIRMED Severity: minor Priority: P3 Component: java AssignedTo: unassigned at gcc dot gnu dot org ReportedBy: dcb314 at hotmail dot com GCC host triplet: suse-linux-x86_64 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38195
[Bug libf2c/38194] New: DefaultListSelectionModel.java: remove dead for loop ?
I just had a look at some of the source code of the Java package in the GNU gcc version 4.4.0 snapshot 20081114 gcc-4.4-20081114/libjava/classpath/javax/swing/DefaultListSelectionModel.java:300 Avoid unused local variables such as 'end'. The source code is int beg = sel.nextSetBit(0), end = -1; for(int i=beg; i >= 0; i=sel.nextSetBit(i+1)) end = i; Suggest remove unused local variable end, since it is in a for loop. It also looks like the for loop can be removed. -- Summary: DefaultListSelectionModel.java: remove dead for loop ? Product: gcc Version: 4.4.0 Status: UNCONFIRMED Severity: minor Priority: P3 Component: libf2c AssignedTo: unassigned at gcc dot gnu dot org ReportedBy: dcb314 at hotmail dot com GCC host triplet: suse-linux-x86_64 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38194
[Bug libstdc++/25191] exception_defines.h #defines try/catch
--- Comment #64 from paolo dot carlini at oracle dot com 2008-11-20 10:24 --- (assuming I understand correctly Jason' approach - didn't really follow in detail the thread, lately) let me know if you want me to remove the exception_defines.h tricks from the library... -- paolo dot carlini at oracle dot com changed: What|Removed |Added CC||paolo at gcc dot gnu dot org http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25191
[Bug fortran/38181] calls to SIZE not optimized out of loops
--- Comment #6 from jakub at gcc dot gnu dot org 2008-11-20 10:21 --- With this patch, I got: $ time ./test1 # 4.4 r142036 real0m3.291s user0m3.289s sys 0m0.002s $ time ./test2 # 4.4 r142037 real0m1.327s user0m1.325s sys 0m0.002s -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38181
[Bug java/38193] New: HashMap.java: unused local variable
I just had a look at some of the source code of the Java package in the GNU gcc version 4.4.0 snapshot 20081114 /home/dcb/gcc/20081114/javaExperiment/gcc-4.4-20081114/libjava/classpath/java/util/HashMap.java:749 Avoid unused local variables such as 'dest'. The source code is HashEntry dest = buckets[idx]; Suggest remove unused local variable dest, since it is in a loop. -- Summary: HashMap.java: unused local variable Product: gcc Version: 4.4.0 Status: UNCONFIRMED Severity: minor Priority: P3 Component: java AssignedTo: unassigned at gcc dot gnu dot org ReportedBy: dcb314 at hotmail dot com GCC host triplet: suse-linux-x86_64 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38193
[Bug java/38192] New: SwingCallbackHandler.java: remove unused local variable
gcc-4.4-20081114/libjava/classpath/gnu/javax/security/auth/callback/SwingCallbackHandler.java:302
Avoid unused local variables such as'defaultIndex'.
The source code is
int defaultIndex = 0;
for (int i = 0; i < locales.length; i++)
{
localeNames[i+1] = locales[i].getDisplayLanguage (locales[i]);
String country = locales[i].getDisplayCountry (locales[i]);
if (country.length () > 0)
localeNames[i+1] += " (" + country + ")";
if (locales[i].equals (locale))
defaultIndex = i;
}
Suggest remove unused local variable defaultIndex, since it is in a loop.
--
Summary: SwingCallbackHandler.java: remove unused local variable
Product: gcc
Version: 4.4.0
Status: UNCONFIRMED
Severity: minor
Priority: P3
Component: java
AssignedTo: unassigned at gcc dot gnu dot org
ReportedBy: dcb314 at hotmail dot com
GCC host triplet: suse-linux-x86_64
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38192
[Bug libstdc++/25191] exception_defines.h #defines try/catch
--- Comment #63 from rguenth at gcc dot gnu dot org 2008-11-20 10:01 --- The patch looks reasonable. I understand that the warning is enabled by default but does not trigger from libstdc++ because that's system headers. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25191
[Bug java/38191] New: gjdoc/Main.java: dead local variable
I just had a look at some of the source code of the Java package in the GNU gcc version 4.4.0 snapshot 20081114 gcc-4.4-20081114/libjava/classpath/tools/gnu/classpath/tools/gjdoc/Main.java:1056 Avoid unused local variables such as 'customOptions'. The source code is List customOptions = new LinkedList(); Suggest remove unused local variable and reduce memory usage at the same time. -- Summary: gjdoc/Main.java: dead local variable Product: gcc Version: 4.4.0 Status: UNCONFIRMED Severity: minor Priority: P3 Component: java AssignedTo: unassigned at gcc dot gnu dot org ReportedBy: dcb314 at hotmail dot com GCC host triplet: suse-linux-x86_64 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38191
[Bug fortran/38181] calls to SIZE not optimized out of loops
--- Comment #5 from jakub at gcc dot gnu dot org 2008-11-20 09:44 --- Subject: Bug 38181 Author: jakub Date: Thu Nov 20 09:42:35 2008 New Revision: 142037 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=142037 Log: PR fortran/38181 * trans-intrinsic.c (gfc_conv_intrinsic_size): Inline 2 argument size if the second argument is not optional and one argument size for rank 1 arrays. * gfortran.dg/array_section_2.f90: Adjust pattern to match the inlined size0 instead of a size0 call. Modified: trunk/gcc/fortran/ChangeLog trunk/gcc/fortran/trans-intrinsic.c trunk/gcc/testsuite/ChangeLog trunk/gcc/testsuite/gfortran.dg/array_section_2.f90 -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38181
[Bug java/38190] New: ConcurrentHashMap.java: 2 * unused local variables
I just had a look at some of the source code of the Java package in the GNU gcc version 4.4.0 snapshot 20081114 gcc-4.4-20081114/libjava/classpath/external/jsr166/java/util/concurrent/ConcurrentHashMap.java:807 Avoid unused local variables such as 'c'. gcc-4.4-20081114/libjava/classpath/external/jsr166/java/util/concurrent/ConcurrentHashMap.java:815 Avoid unused local variables such as 'c'. I have checked the source code and I agree. Suggest remove these two unused local variables, especially since they are in a for loop. -- Summary: ConcurrentHashMap.java: 2 * unused local variables Product: gcc Version: 4.4.0 Status: UNCONFIRMED Severity: minor Priority: P3 Component: java AssignedTo: unassigned at gcc dot gnu dot org ReportedBy: dcb314 at hotmail dot com GCC host triplet: suse-linux-x86_64 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38190
[Bug java/38189] New: Java: set but not used local variable in tight loop
I just had a look at some of the source code of the Java package in the GNU gcc version 4.4.0 snapshot 20081114 gcc-4.4-20081114/libjava/classpath/gnu/xml/aelfred2/XmlParser.java:3205 Avoid unused local variables such as 'ampRead'. I have checked the source code and I agree. Suggest remove set but not used local variable, especially since it is written to for each character read. -- Summary: Java: set but not used local variable in tight loop Product: gcc Version: 4.4.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: java AssignedTo: unassigned at gcc dot gnu dot org ReportedBy: dcb314 at hotmail dot com GCC host triplet: suse-linux-x86_64 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38189
[Bug c/38187] when using gcc compile the code with option "-g3", I find the inline assemble code are palced in section .debug_macinfo
--- Comment #1 from schwab at suse dot de 2008-11-20 09:19 --- *** This bug has been marked as a duplicate of 38186 *** -- schwab at suse dot de changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution||DUPLICATE http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38187
[Bug c/38186] when using gcc compile the code with option "-g3", I find the inline assemble code are palced in section .debug_macinfo
--- Comment #1 from schwab at suse dot de 2008-11-20 09:19 --- *** Bug 38187 has been marked as a duplicate of this bug. *** -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38186
[Bug fortran/38181] calls to SIZE not optimized out of loops
--- Comment #4 from jv244 at cam dot ac dot uk 2008-11-20 08:53 --- so I guess this is up to the front end to generate 'better' code for size -- jv244 at cam dot ac dot uk changed: What|Removed |Added OtherBugsDependingO||36854 nThis|| http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38181
[Bug tree-optimization/32306] [4.2/4.3/4.4 Regression] redundant && || not eliminated
--- Comment #13 from bonzini at gnu dot org 2008-11-20 08:48 --- I agree with Steven that the bug title does not make sense. It is the same as complaining that IRA has a regression because it disables regmove. OTOH, this *is* a regression and the bug should stay open. For example, it could be fixed by doing some of the tree optimizations with && not lowered to jumps (just shooting). -- bonzini at gnu dot org changed: What|Removed |Added CC||bonzini at gnu dot org Summary|[4.2/4.3/4.4 Regression] DOM|[4.2/4.3/4.4 Regression] |jump threading no longer|redundant && || not |iterates|eliminated http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32306
[Bug rtl-optimization/32283] [4.3/4.4 regression] Missed induction variable optimization
--- Comment #23 from rakdver at gcc dot gnu dot org 2008-11-20 08:06 --- Subject: Bug 32283 Author: rakdver Date: Thu Nov 20 08:05:12 2008 New Revision: 142035 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=142035 Log: PR rtl-optimization/32283 * tree-ssa-loop-niter.c (scev_probably_wraps_p): Use type of the base of the induction variable to decide whether it may wrap. * tree-ssa-loop-ivopts.c (rewrite_use_compare): Emit the initialization of the bound before the loop. * simplify-rtx.c (simplify_binary_operation_1): Add two simplifications regarding AND. (simplify_plus_minus): Only fail if no simplification is possible. * loop-iv.c (simple_rhs_p): Consider reg + reg and reg << cst simple. Modified: trunk/gcc/ChangeLog trunk/gcc/loop-iv.c trunk/gcc/simplify-rtx.c trunk/gcc/tree-ssa-loop-ivopts.c trunk/gcc/tree-ssa-loop-niter.c -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32283
