Sorry for the delay, folks – we did some pipecleaning of some changes that conflicted with this. I’ve pushed the following change, which should enable this:

https://review.trustedfirmware.org/c/TF-A/trusted-firmware-a/+/26693

Chris

From: Chris Kay via TF-A <tf-a@lists.trustedfirmware.org>
Date: Monday, 5 February 2024 at 14:27
To: Michal Simek <michal.simek@amd.com>, Kummari, Prasad <Prasad.Kummari@amd.com>, tf-a@lists.trustedfirmware.org <tf-a@lists.trustedfirmware.org>
Cc: Nagal, Amit <amit.nagal@amd.com>, Belsare, Akshay <akshay.belsare@amd.com>
Subject: [TF-A] Re: ENABLE_LTO feature usage.

I’ve triggered CI+2 on the LTO-by-default patch, but it’s encountering build failures already: https://ci.trustedfirmware.org/job/tf-a-gerrit-tforg-l2/1689/

I’ll try to find some time to move the LTO configuration after platform file inclusion later this week, unless anybody else is willing to volunteer to get it sorted faster.

Chris

From: Michal Simek <michal.simek@amd.com>
Date: Monday, 5 February 2024 at 12:21
To: Chris Kay <Chris.Kay@arm.com>, Kummari, Prasad <Prasad.Kummari@amd.com>, tf-a@lists.trustedfirmware.org <tf-a@lists.trustedfirmware.org>
Cc: Nagal, Amit <amit.nagal@amd.com>, Belsare, Akshay <akshay.belsare@amd.com>
Subject: Re: ENABLE_LTO feature usage.

On 2/5/24 13:04, Chris Kay wrote:
> Assuming it still fails to pass then we could perhaps move the LTO check
> somewhere after the platform file is included and hope for the best. It’s a bit
> spaghetti and it would break any platforms that read ENABLE_LTO, but I don’t
> think there are any so cross fingers and hope for the best.
>
> With that said, we also ought to address any failures anyway because they are
> generally indicative of UB or broken linker scripts.

I am reading it that you will do this step and keep us in CC and we are happy to
test and help and we will definitely set it up for that one configuration.

Thanks,
Michal