[PATCH] D82200: [ADT] Fix itostr handling of min int64_t value
Thomas Preud'homme via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 22 06:24:59 PDT 2020
thopre added a comment.
In D82200#2105987 <https://reviews.llvm.org/D82200#2105987>, @jhenderson wrote:
> Shouldn't there be some unit testing for this? It seems like a better testing method than relying on an UBSan build bot failure that might not manifest again in the future if some other testing were changed.
Will do, mind if I add you as reviewer?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82200/new/
https://reviews.llvm.org/D82200
More information about the llvm-commits
mailing list