Blind attempt to fix LLVM dependency in the backend
authorJohn Naylor <john.naylor@postgresql.org>
Thu, 15 Sep 2022 03:24:55 +0000 (10:24 +0700)
committerJohn Naylor <john.naylor@postgresql.org>
Thu, 15 Sep 2022 03:53:48 +0000 (10:53 +0700)
Commit ecaf7c5df5 removed gram.h from the backend's generated-headers
target. In LLVM builds, this leads to loss of dependency information
when generating .bc files. To fix, add a rule that mirrors ad-hoc .o
dependencies for .bc files as well.

Per cfbot (no buildfarm failures reported)
Analysis by Tom Lane and Andres Freund
Proposed fix by Andres Freund
Discussion: https://www.postgresql.org/message-id/20220914210427.y26tkagmxo5wwbvp%40awork3.anarazel.de

src/backend/common.mk

index fa96a82b1a0cc4a1b004877dc0e12d24e1c0fa87..61861f5c7ebda50bb003651a4379402834140ef6 100644 (file)
@@ -23,6 +23,7 @@ objfiles.txt: Makefile $(SUBDIROBJS) $(OBJS)
 
 ifeq ($(with_llvm), yes)
 objfiles.txt: $(patsubst %.o,%.bc, $(OBJS))
+$(patsubst %.o,%.bc, $(OBJS)): $(OBJS)
 endif
 
 # make function to expand objfiles.txt contents