suoyuan666 wrote:

> This looks pretty good to me! Though I am wondering whether we actually need 
> all this debug stuff in `buildOriginFlowChain`.

Thanks! I originally added these debug statements to help explain my previous 
design. They also make it easier to correlate the output of 
`FactManager.dump()` with `OriginID` found by `buildOriginFlowChain` and to 
understand how they relate to the later diagnostic output.

After simplifying the search logic, however, they are no longer necessary for 
explaining how this function works...

https://github.com/llvm/llvm-project/pull/204592
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to