ilovepi wrote: > I think having a separate unittest target is also perfectly reasonable to do > in a follow up patch.
Right. I'm not suggesting its a requirement to land. just an observation that we probably should. https://github.com/llvm/llvm-project/pull/165935 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
