On 09/03/2020 15:10, Jan Beulich wrote: > On 09.03.2020 16:07, Tamas K Lengyel wrote: >> On Mon, Mar 9, 2020 at 5:49 AM Jan Beulich <jbeul...@suse.com> wrote: >>> The common header doesn't itself need to include public/vm_event.h nor >>> public/memory.h. Drop their inclusion. This requires using the non- >>> typedef names in two prototypes and an inline function; by not changing >>> the callers and function definitions at the same time it'll remain >>> certain that the build would fail if the typedef itself was changed. >> Just curious, what's the benefit of doing this? > Less dependencies that (almost) every CU gets, and hence statistically > less rebuilds of (almost) everything when only a rather special purpose > header file changes.
Also fractionally faster compile times generally (reduced parsing of unrelated header files), and overall, a reduction in the complexity of the tangled mess that is our header files. ~Andrew _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
