[Bug build/28922] The .d dependency files aren't always generated

[skpgkp2 at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=28922

Sunil Pandey <skpgkp2 at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |goldstein.w.n at gmail dot com,
                   |                            |skpgkp2 at gmail dot com

--- Comment #3 from Sunil Pandey <skpgkp2 at gmail dot com> ---
c12c2a41b0 benchtests: Generate .d dependency files [BZ #28922] breaks libmvec
benchmark build and silently ignore it.

libmvec benchmark used to build with "make bench" command, but now it ignores
it.

Reverting Rule line to its previous location fixes the libmvec benchmark issue.


$ cat 0001-benchtests-Revert-Rules-include-line-to-its-previous.patch
>From 87e6cdd5cee293c178982e486eeeac3343e1d5ff Mon Sep 17 00:00:00 2001
From: Sunil K Pandey <skpgkp2@gmail.com>
Date: Tue, 15 Mar 2022 18:52:19 -0700
Subject: [PATCH] benchtests: Revert Rules include line to its previous
 location.

---
 benchtests/Makefile | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/benchtests/Makefile b/benchtests/Makefile
index d513b29903..873bbe3fa8 100644
--- a/benchtests/Makefile
+++ b/benchtests/Makefile
@@ -296,6 +296,8 @@ bench-extra-objs = json-lib.o
 extra-objs += $(bench-extra-objs)
 others-extras = $(bench-extra-objs)

+include ../Rules
+
 binaries-bench := $(addprefix $(objpfx)bench-,$(bench))
 extra-objs += $(addsuffix .o,$(addprefix bench-,$(bench)))
 binaries-benchset := $(addprefix $(objpfx)bench-,$(benchset))
@@ -337,8 +339,6 @@ run-bench = $(test-wrapper-env) \
 timing-type := $(objpfx)bench-timing-type
 extra-objs += bench-timing-type.o

-include ../Rules
-
 # This makes sure CPPFLAGS-nonlib and CFLAGS-nonlib are passed
 # for all these modules.
 cpp-srcs-left := $(binaries-benchset:=.c) $(binaries-bench:=.c) \
-- 
2.35.1

-- 
You are receiving this mail because:
You are on the CC list for the bug.