Re: [PATCH 0/4] Fix stack usage of DML

2021-09-15 Thread Leo Li
On 2021-09-08 21:00, Harry Wentland wrote: With the '-Werror' enablement patch the amdgpu build was failing on clang builds because a bunch of functions were blowing past the 1024 byte stack frame default. Due to this we also noticed that a lot of functions were passing large structs by value

Re: [PATCH 0/4] Fix stack usage of DML

2021-09-09 Thread Nathan Chancellor
On Wed, Sep 08, 2021 at 09:00:19PM -0400, Harry Wentland wrote: > With the '-Werror' enablement patch the amdgpu build was failing > on clang builds because a bunch of functions were blowing past > the 1024 byte stack frame default. Due to this we also noticed > that a lot of functions were

Re: [PATCH 0/4] Fix stack usage of DML

2021-09-09 Thread Christian König
It's nice to see at least some of them addressed, feel free to add an Acked-by: Christian König Regards, Christian. Am 09.09.21 um 03:00 schrieb Harry Wentland: With the '-Werror' enablement patch the amdgpu build was failing on clang builds because a bunch of functions were blowing past the

[PATCH 0/4] Fix stack usage of DML

2021-09-08 Thread Harry Wentland
With the '-Werror' enablement patch the amdgpu build was failing on clang builds because a bunch of functions were blowing past the 1024 byte stack frame default. Due to this we also noticed that a lot of functions were passing large structs by value instead of by pointer. This series attempts to

[PATCH 0/4] Fix stack usage of DML

2021-09-08 Thread Harry Wentland
With the '-Werror' enablement patch the amdgpu build was failing on clang builds because a bunch of functions were blowing past the 1024 byte stack frame default. Due to this we also noticed that a lot of functions were passing large structs by value instead of by pointer. This series attempts to