[PATCH] D61165: Fix a crash where a [[no_destroy]] destructor was not emitted in an array
    John McCall via Phabricator via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Fri May  3 16:46:49 PDT 2019
    
    
  
rjmccall added a comment.
The flip side of that argument is that (1) there aren't very many users right now and (2) it's much easier to start conservative and then weaken the rule than it will be to strengthen it later.  It really isn't acceptable to just turn off access/use-checking for the destructor, so if we get trapped by the choice we make here, we'll end up having to either leak or call `std::terminate`.
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61165/new/
https://reviews.llvm.org/D61165
    
    
More information about the cfe-commits
mailing list