YexuanXiao wrote:

> Maybe split out the type sugar part from the warning part when you resubmit? 
> We could merge the type sugar part back straight away, while the warnings are 
> improved.

I'll relax the checks to prevent false positives (only reporting UB and no 
longer reporting non-portable). This should be the simplest approach for now, 
avoiding the need to separate the two parts of the code again. Even if we only 
keep the type sugar, we'll still need to adjust the format string checking code 
because some tests need to recognize the new sugar types, that was actually why 
I modified it in the first place.  


https://github.com/llvm/llvm-project/pull/143653
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to