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