svn commit: r293724 - in head/sys/boot: arm64/libarm64 common efi/boot1 efi/fdt efi/include efi/include/arm64 efi/libefi efi/loader efi/loader/arch/amd64 efi/loader/arch/arm efi/loader/arch/arm64 i...

[Steven Hartland]

On 13/01/2016 01:10, Allan Jude wrote:
> On 2016-01-12 20:05, NGie Cooper wrote:
>> On Tue, Jan 12, 2016 at 4:54 PM, Ian Lepore <ian at freebsd.org> wrote:
>> ...
>>> For several reasons.  The fact that gcc isn't the default compiler
>>> doesn't mean that it's okay for code to not compile with gcc; it's
>>> still a supported compiler for arm.
>>
>> Agreed. It also makes things messier when MFCing changes if it doesn't
>> work with gcc.
>> Thanks,
>> -NGie
>>
>
> If it is a supported compiler, maybe we need a tinderbox option that 
> compiles ARM with gcc, to catch cases like this.
>
Yer we really should.

I explicitly ran tinderbox on these changes in order to ensure all the 
bases where covered, if there are additional options not run by 
tinderbox but are supported then we really need add them otherwise we're 
just assuming everyone knows to manually test additional configurations 
which is never going to work.

     Regards
     Steve