Bug 468606 - build: remove "Valgrind relies on GCC" check/output
Summary: build: remove "Valgrind relies on GCC" check/output
Status: RESOLVED FIXED
Alias: None
Product: valgrind
Classification: Developer tools
Component: general (show other bugs)
Version: 3.21 GIT
Platform: unspecified Unspecified
: NOR normal
Target Milestone: ---
Assignee: Julian Seward
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-04-17 10:49 UTC by fanquake
Modified: 2023-04-18 20:28 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
0001-build-remove-Valgrind-relies-on-GCC-check-output.patch (1.01 KB, application/mbox)
2023-04-18 08:14 UTC, fanquake
Details

Note You need to log in before you can comment on or make changes to this bug.
Description fanquake 2023-04-17 10:49:45 UTC
This code was added in 2002, before Clang even existed. The 'Valgrind
relies on GCC" output is certainly no-longer true, and this check
"passes" when using vanilla Clang, or Clang pretending to be GCC etc.

Seems easiest to just remove this code, as there are numerous other
compiler checks throughout configure.

Patch to remove is attached.
Comment 1 Paul Floyd 2023-04-17 19:26:10 UTC
You didn't add the patch. Do you mean removing

# Check for the compiler support
if test "${GCC}" != "yes" ; then
   AC_MSG_ERROR([Valgrind relies on GCC to be compiled])
fi

?
Comment 2 Paul Floyd 2023-04-17 19:54:58 UTC
commit 41a7f59a8838a042813ac20fe1472e55e9bd5697 (HEAD -> master, origin/master, origin/HEAD)
Author: Paul Floyd <pjfloyd@wanadoo.fr>
Date:   Mon Apr 17 21:53:23 2023 +0200

    Bug 374596 - inconsistent RDTSCP support on x86_64
Comment 3 fanquake 2023-04-18 08:14:10 UTC
Created attachment 158183 [details]
0001-build-remove-Valgrind-relies-on-GCC-check-output.patch
Comment 4 fanquake 2023-04-18 08:14:43 UTC
> You didn't add the patch. Do you mean removing

🤦 Sorry. Yes, those lines. Patch attached now in any case.
Comment 5 Paul Floyd 2023-04-18 20:28:48 UTC
commit 1e784548a17aec105ed79d53c91c3fd023b2c364 (HEAD -> master, origin/master, origin/HEAD)
Author: Paul Floyd <pjfloyd@wanadoo.fr>
Date:   Tue Apr 18 22:27:55 2023 +0200

    Bug 468606 - build: remove "Valgrind relies on GCC" check/output