[PATCH] D50512: [MS Demangler] Fix several issues related to templates
    Reid Kleckner via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Thu Aug  9 18:20:04 PDT 2018
    
    
  
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
lgtm
================
Comment at: llvm/lib/Demangle/MicrosoftDemangle.cpp:1371
+  auto Number = demangleNumber(MangledName);
+  assert(!Number.second);
 
----------------
This assert is implied by startsWithLocalScopePattern and can't be tripped by corrupt input, right?
https://reviews.llvm.org/D50512
    
    
More information about the llvm-commits
mailing list