[PATCH] D125061: [clang] A more robust way to attach comments
    Zixu Wang via Phabricator via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Thu May  5 19:12:26 PDT 2022
    
    
  
zixuw added a comment.
Well, apparently there are still some corner-case bugs in the logic. ExtractAPI and a few other tests failed. Will look into it tomorrow.
And if folks have concerns or opinions of the performance/space cost of adding the extra pointer to `DeclBase`, I can try to keep the original logic and just special-case enum constants to find the previous enum constant decl in the parent enum. Not sure if there are existing ways to get preceding enum constants efficiently, otherwise it'd be an O(n^2) operation for an enum.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D125061/new/
https://reviews.llvm.org/D125061
    
    
More information about the cfe-commits
mailing list