Re: [PATCH] BaseTools/tools_def: use separate PP definition for DTC

[Ard Biesheuvel <ard.biesheuvel@linaro.org>]

On 27 February 2018 at 18:33, Leif Lindholm <leif.lindholm@linaro.org> wrote:
> On Tue, Feb 27, 2018 at 05:51:32PM +0000, Ard Biesheuvel wrote:
>> Clang's preprocessor behaves differently from GCC's, and produces
>> intermediate device tree source that still contains #pragma pack()
>> and other directives that the device tree compiler chokes on.
>>
>> For assembling device tree sources, it matters very little which
>> preprocessor is being used, so let's just use GNU CPP explicitly.
>
> Ah, right, same fundamental issue as
> 5b8766bb92debfa7b2f45a4a6d683b4227360d66.
>

Yes, and I fail to see why changing just those two files makes a
meaningful difference.

> However, this time triggered by autogen seemingly forcing inclusion of
> lots of central header files that are not even used, like
> MdePkg/Include/IndustryStandard/Bluetooth.h
> MdePkg/Include/IndustryStandard/Acpi10.h
> and so on.
>
> Is there any way to suppress these implicit includes from .dts
> processing?
>

There is 'Trim', which can filter #include'd content, but using that
also robs us of the ability to #include .dtsi files, which was kind of
the point of using the preprocessor in the first place.