[PATCH] D126197: [analyzer] SATest: Weaken assumption about HTML files
Marco Antognini via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue May 24 02:27:01 PDT 2022
mantognini added a comment.
In D126197#3533674 <https://reviews.llvm.org/D126197#3533674>, @steakhal wrote:
> I'm not using this script file, so I'm not gonna be too picky about this.
> However, I would be more confident landing changes if we had tests exercising the changed behavior.
> If you plan to land more changes to this file, consider adding some tests.
I agree tests would be good, but I'm not sure where to start. To clarify, that is also the only change I've got to this file and I don't plan further change in the near future.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D126197/new/
https://reviews.llvm.org/D126197
More information about the cfe-commits
mailing list