[PATCH] D123155: [analyzer] Expose Taint.h to plugins

This revision was automatically updated to reflect the committed changes. Closed by commit rG82f3ed99045d: [analyzer] Expose Taint.h to plugins (authored by tomrittervg, committed by steakhal). Changed prior to commit: https://reviews.llvm.org/D123155?vs=420616&id=423622#toc Repository: rG L

[PATCH] D123155: [analyzer] Expose Taint.h to plugins

tomrittervg added a comment. Please commit on my behalf; you can use "Tom Ritter " Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D123155/new/ https://reviews.llvm.org/D123155 ___ cfe-commits mailing list

[PATCH] D123155: [analyzer] Expose Taint.h to plugins

steakhal added a comment. In D123155#3433715 , @tomrittervg wrote: > Just a note - I was able to test this finally and it works for what I was > trying to do. Do you have commit access? Or we shall commit this on your behalf? If the latter, please tell

[PATCH] D123155: [analyzer] Expose Taint.h to plugins

tomrittervg added a comment. Just a note - I was able to test this finally and it works for what I was trying to do. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D123155/new/ https://reviews.llvm.org/D123155 __

[PATCH] D123155: [analyzer] Expose Taint.h to plugins

steakhal accepted this revision. steakhal added a comment. LGTM. Please note that we dont guarantee abi nor api compatibility. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D123155/new/ https://reviews.llvm.org/D123155 _

[PATCH] D123155: [analyzer] Expose Taint.h to plugins

NoQ accepted this revision. NoQ added a comment. Looks good. I never intended taint to be inaccessible from plugins. Probably there are other headers in `lib/StaticAnalyzer/Checkers` that deserve the same treatment. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://rev

[PATCH] D123155: [analyzer] Expose Taint.h to plugins

xazax.hun accepted this revision. xazax.hun added a comment. This revision is now accepted and ready to land. The changes look good to me but please wait at least one more reviewer before committing. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D12

[PATCH] D123155: [analyzer] Expose Taint.h to plugins

xazax.hun added reviewers: NoQ, steakhal, Szelethus, martong, xazax.hun. xazax.hun added a comment. Herald added a subscriber: rnkovacs. Adding some reviewers Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D123155/new/ https://reviews.llvm.org/D12315

[PATCH] D123155: [analyzer] Expose Taint.h to plugins

tomrittervg created this revision. Herald added subscribers: manas, steakhal, ASDenysPetrov, martong, dkrupp, donat.nagy, Szelethus, mikhail.ramalho, a.sidorin, szepet, baloghadamsoftware, xazax.hun. Herald added a project: All. tomrittervg requested review of this revision. Herald added a projec