Re: [PATCH v4] Add strict aliasing warning when inlining function.

2014-08-21 Thread lin zuojian
Hi, My patch for warning about strict aliasing violation is not strict enough. Current implementation of strict_aliasing_warn has not consider tbaa. So a test case gcc/testsuite/g++.dg/opt/pmf1.C will emit additional warning whereas it shouldn't do. My patch is not good for comm

Re: [PATCH v4] Add strict aliasing warning when inlining function.

2014-08-21 Thread lin zuojian
Hi, Currently warning about strict aliasing is not strict enough. It has not considered tbaa. So a test case emit addition error. This patch is not good for commit. -- Lin Zuojian

Re: [PATCH v4] Add strict aliasing warning when inlining function.

2014-08-20 Thread lin zuojian
Hi Richard, Do you think this version is good enough to commit to trunk? I have asked Andrew to commit for me. If it is not good enough, I may ask him not to do this. --- Lin Zuojian

Re: [PATCH v4] Add strict aliasing warning when inlining function.

2014-08-20 Thread lin zuojian
Hi Andrew, What do you think of the current patch? If you thought it was find to commit to trunk, please commit it for me. -- Lin Zuojian

[PATCH v4] Add strict aliasing warning when inlining function.

2014-08-20 Thread lin zuojian
Hi, This patch is to improve the output of strict_aliasing_warning. This version will output the expression of rhs, type of that expression, and the type of lhs. That make it easier to debug strict aliasing bug. * tree-inline.c (setup_one_parameter): Add strict aliasing check.