Messages in this thread | | | Date | Thu, 4 Aug 2022 15:43:48 -0700 | From | Nathan Chancellor <> | Subject | Re: mainline build failure for x86_64 allmodconfig with clang |
| |
On Thu, Aug 04, 2022 at 02:59:01PM -0700, Linus Torvalds wrote: > On Thu, Aug 4, 2022 at 1:43 PM Nathan Chancellor <nathan@kernel.org> wrote: > > > > I do note that commit 1b54a0121dba ("drm/amd/display: Reduce stack size > > in the mode support function") did have a workaround for GCC. It appears > > clang will still inline mode_support_configuration(). If I mark it as > > 'noinline', the warning disappears in that file. > > That sounds like probably the best option for now. Gcc does not inline > that function (at least for allmodconfig builds in my testing), so if > that makes clang match what gcc does, it seems a reasonable thing to > do.
Sounds good. That solution only takes care of the warning in display_mode_vba_32.c. I will try and come up with something similar for the other two files tomorrow, unless the AMD folks beat me to it, since they will know the driver better than I will ;)
Cheers, Nathan
| |