mgehre added inline comments.
================ Comment at: clang/include/clang/Analysis/FlowSensitive/DataflowWorklist.h:20 +namespace clang { +template <typename Comp, unsigned QueueSize> class DataflowWorklistBase { + llvm::BitVector EnqueuedBlocks; ---------------- xazax.hun wrote: > xazax.hun wrote: > > mgehre wrote: > > > Should this class have a bit of doxygen and a unit test? > > We have two users and both users have regression tests. More tests are > > always good, but I am not sure if we would get much value in this case. > > Having some comments sound very useful though :) > Added a unit test anyway :) ;-) ================ Comment at: clang/include/clang/Analysis/FlowSensitive/DataflowWorklist.h:20 +namespace clang { +/// A worklist implementation where the enqueued blocks wwill be dequeued based +/// on the order defined by 'Comp'. ---------------- nit: wwill CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72380/new/ https://reviews.llvm.org/D72380 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits