lslimani wrote: > > > But I am wondering if we can use this in canReuse too. > > > > Well ig we could, but I'm a bit worried about perfs on projects with lots > > of modules. > > I think at least this should be faster than the current mechanism.
Alone yes it is, and I think by a lot. The thing is that, I'm not sure but I think that some cases that don't need to be handled in hasNewerDependnecies, do have to be handled in canReuse, thus we would have to include both checks. But I agree this should be looked into further to be sure. https://github.com/llvm/llvm-project/pull/234181 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
