[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