aganea added a comment.

In D72982#1831595 <https://reviews.llvm.org/D72982#1831595>, @hans wrote:

> Wait, do we really want the "(in-process)" marker to be written to a separate 
> line? I'm not sure that we do.


Do you see value in keeping "(in-process)" at all? I added it in the first 
place for testing.

> The change description doesn't explain how scan-build was broken or how this 
> fixes it.

Fixed in the summary above. Should I revert the patch and recommit with a 
proper description/proper fix?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D72982/new/

https://reviews.llvm.org/D72982



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to