[PATCH] D114095: [clang][lex] Include tracking: simplify and move to preprocessor

[Volodymyr Sapsai via Phabricator via cfe-commits]

vsapsai added a comment.

I've mentioned it in D112915 <https://reviews.llvm.org/D112915> as we've discussed the stored data format there. But my concern was that bitvector packing might be not the most space-efficient encoding. I haven't done proper testing, just off-the-cuff comparison and it looks like for the most of frameworks in iOS SDK storing included headers per submodule takes less space than encoding them as a bitvector. I have an idea why that might be happening but I haven't checked it in debugger, so'll keep it to myself to avoid derailing the discussion.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D114095/new/

https://reviews.llvm.org/D114095