aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land.
This looks.... good?.... to me. :-) Despite this complicating things by a fair amount, I don't have a better suggestion to offer. LGTM ================ Comment at: clang/lib/ASTMatchers/ASTMatchFinder.cpp:771 +// We don't have enough free low bits in 32bit builds to discriminate 8 pointer +// types in PointerUnion. so split the union in 2 using a free bit from the +// callback pointer. ---------------- Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122529/new/ https://reviews.llvm.org/D122529 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
