Another ping on the Middle-end review of this patch. 

This patch has been waiting for the middle-end review for a long time. 

Please review it and provide any feedback, I believe that this should be a nice 
improvement to GCC diagnostic in general. 

Thanks.

Qing

> On Nov 15, 2024, at 10:34, Qing Zhao <qing.z...@oracle.com> wrote:
> 
> Gentle ping on the middle-end review for this patch. 
> 
> There are two parts of this patch:
> 
> 1. Diagnostic part (Part 2), which has been reviewed by David;
> 2. Middle end part (Part 1 and 3), mainly on the copy_history information 
> collection during transformation. 
> 
> Thanks,
> 
> Qing
> 
> 
>> On Nov 5, 2024, at 11:31, Qing Zhao <qing.z...@oracle.com> wrote:
>> 
>> Hi,
>> 
>> This is the 4th version of the patch for fixing PR109071.
>> 
>> Compared to the 3nd version:
>> https://gcc.gnu.org/pipermail/gcc-patches/2024-October/666870.html
>> https://gcc.gnu.org/pipermail/gcc-patches/2024-October/666872.html
>> https://gcc.gnu.org/pipermail/gcc-patches/2024-October/666871.html
>> 
>> The major improvements to this patch are:
>> 
>> 1. Divide the patch into 3 parts:
>>   Part 1: Add new data structure move_history, record move_history during
>>           transformation;
>>   Part 2: In warning analysis, Use the new move_history to form a rich
>>           location with a sequence of events, to report more context info
>>           of the warnings.
>>   Part 3: Add debugging mechanism for move_history.
>> 
>> 2. Major change to the above Part 2, completely rewritten based on David's
>>  new class lazy_diagnostic_path. 
>> 
>> 3. Fix all issues identied By Sam;
>>  A. fix PR117375 (Bug in tree-ssa-sink.cc);
>>  B. documentation clarification;
>>  C. Add all the duplicated PRs in the commit comments;
>> 
>> 4. Bootstrap GCC with the new -fdiagnostics-details on by default (Init (1)).
>>  exposed some ICE similar as PR117375 in tree-ssa-sink.cc, fixed.
>> 
>> 
>> bootstrapping and regression testing on both x86 and aarch64.
>> 
>> Please let me know any comment and suggestion.
>> 
>> Thanks.
>> 
>> Qing
>> Qing Zhao (3):
>> Provide more contexts for -Warray-bounds, -Wstringop-* warning
>>   messages due to code movements from compiler transformation (Part 1)
>>   [PR109071,PR85788,PR88771,PR106762,PR108770,PR115274,PR117179]
>> Provide more contexts for -Warray-bounds, -Wstringop-* warning
>>   messages due to code movements from compiler transformation (Part 2)
>>   [PR109071,PR85788,PR88771,PR106762,PR108770,PR115274,PR117179]
>> Provide more contexts for -Warray-bounds, -Wstringop-* warning
>>   messages due to code movements from compiler transformation (Part 3)
>>   [PR109071,PR85788,PR88771,PR106762,PR108770,PR115274,PR117179]
>> 
>> 
> 

Reply via email to