Re: [PATCH] selftests: default to host arch for LLVM builds

2024-05-03 Thread Shuah Khan
On 4/28/24 06:08, Valentin Obst wrote: Align the behavior for gcc and clang builds by interpreting unset `ARCH` and `CROSS_COMPILE` variables in `LLVM` builds as a sign that the user wants to build for the host architecture. This patch preserves the properties that setting the `ARCH` variable to

Re: [PATCH] selftests: default to host arch for LLVM builds

2024-04-30 Thread John Hubbard
On 4/30/24 7:50 AM, Mark Brown wrote: On Tue, Apr 30, 2024 at 01:44:52PM +0200, Valentin Obst wrote: On 4/29/24 12:04 AM, John Hubbard wrote: Correct me if I'm wrong, but intuitively 30+ patches that touch selftests from many different subsystems do not sound like something that is going to be m

Re: [PATCH] selftests: default to host arch for LLVM builds

2024-04-30 Thread Mark Brown
On Tue, Apr 30, 2024 at 01:44:52PM +0200, Valentin Obst wrote: > On 4/29/24 12:04 AM, John Hubbard wrote: > > I'm expecting to post the series soon, hopefully this week. And I'm > > thinking maybe I should carry your patch as the first one in the series, > > in order to ensure it gets picked up. O

Re: [PATCH] selftests: default to host arch for LLVM builds

2024-04-30 Thread Valentin Obst
On 4/29/24 12:04 AM, John Hubbard wrote: > ... >>> And yes, the selftests "normal" (non-cross-compile) build is *broken* >>> right now, for clang. I didn't realize from the patch title that this is >>> actually a significant fix. Maybe we should change the subject line >>> (patch >>> title) to some

Re: [PATCH] selftests: default to host arch for LLVM builds

2024-04-28 Thread John Hubbard
On 4/28/24 5:08 AM, Valentin Obst wrote: ... And yes, the selftests "normal" (non-cross-compile) build is *broken* right now, for clang. I didn't realize from the patch title that this is actually a significant fix. Maybe we should change the subject line (patch title) to something like: [P

Re: [PATCH] selftests: default to host arch for LLVM builds

2024-04-28 Thread Valentin Obst
> > Align the behavior for gcc and clang builds by interpreting unset > > `ARCH` and `CROSS_COMPILE` variables in `LLVM` builds as a sign that the > > user wants to build for the host architecture. > > > > This patch preserves the properties that setting the `ARCH` variable to an > > unknown value

Re: [PATCH] selftests: default to host arch for LLVM builds

2024-04-13 Thread John Hubbard
On 3/29/24 3:49 AM, Valentin Obst wrote: Align the behavior for gcc and clang builds by interpreting unset `ARCH` and `CROSS_COMPILE` variables in `LLVM` builds as a sign that the user wants to build for the host architecture. This patch preserves the properties that setting the `ARCH` variable

[PATCH] selftests: default to host arch for LLVM builds

2024-03-29 Thread Valentin Obst
Align the behavior for gcc and clang builds by interpreting unset `ARCH` and `CROSS_COMPILE` variables in `LLVM` builds as a sign that the user wants to build for the host architecture. This patch preserves the properties that setting the `ARCH` variable to an unknown value will trigger an error t