Skip to content

Commit 5f1d644

Browse files
authored
Merge pull request #1041 from roiser/change-amd-arch
Allow setting the AMD GPU architecture from outside the makefile
2 parents 17e12ca + ad13e26 commit 5f1d644

File tree

3 files changed

+5
-3
lines changed

3 files changed

+5
-3
lines changed

MG5aMC/mg5amcnlo

epochX/cudacpp/CODEGEN/PLUGIN/CUDACPP_SA_OUTPUT/madgraph/iolibs/template_files/gpu/cudacpp.mk

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -229,6 +229,8 @@ ifeq ($(BACKEND),cuda)
229229

230230
else ifeq ($(BACKEND),hip)
231231

232+
# example architecture values MI200:gfx90a, MI350X:gfx942
233+
MADGRAPH_HIP_ARCHITECTURE ?= gfx942
232234
# Set GPUCC as $(HIP_HOME)/bin/hipcc (it was already checked above that this exists)
233235
GPUCC = $(HIP_HOME)/bin/hipcc
234236
XCOMPILERFLAG =
@@ -243,7 +245,7 @@ else ifeq ($(BACKEND),hip)
243245
###GPUFLAGS += -ggdb # FOR DEBUGGING ONLY
244246

245247
# AMD HIP architecture flags
246-
GPUARCHFLAGS = --offload-arch=gfx942
248+
GPUARCHFLAGS = --offload-arch=${MADGRAPH_HIP_ARCHITECTURE}
247249
GPUFLAGS += $(GPUARCHFLAGS)
248250

249251
# Other AMD-specific flags

epochX/cudacpp/CODEGEN/PLUGIN/CUDACPP_SA_OUTPUT/madgraph/iolibs/template_files/gpu/cudacpp_test.mk

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ CXXFLAGS += -Igoogletest/googletest/include/ -std=c++11
1919
all: googletest/$(INSTALLDIR)/lib64/libgtest.a
2020

2121
googletest/CMakeLists.txt:
22-
git clone https://github.com/google/googletest.git -b release-1.11.0 googletest
22+
git clone https://github.com/google/googletest.git -b v1.17.0 googletest
2323

2424
googletest/$(BUILDDIR)/Makefile: googletest/CMakeLists.txt
2525
mkdir -p googletest/$(BUILDDIR)

0 commit comments

Comments
 (0)