[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