From 73a78baf81220a7b1fd34e9ddbdb54f5dd73919a Mon Sep 17 00:00:00 2001 From: Sergio Afonso Date: Wed, 14 Aug 2024 11:54:24 +0100 Subject: [PATCH] [OpenMP][OMPIRBuilder] Remove redundant initialization of IsTargetDevice This property of the `OpenMPIRBuilderConfig` is already initialized in the constructor for `OMPInformationCache`. This looks to be a leftover after a merge from main (comment [here](https://github.com/llvm/llvm-project/pull/67000#discussion_r1366908227)). --- llvm/lib/Transforms/IPO/OpenMPOpt.cpp | 1 - 1 file changed, 1 deletion(-) diff --git a/llvm/lib/Transforms/IPO/OpenMPOpt.cpp b/llvm/lib/Transforms/IPO/OpenMPOpt.cpp index 9790413ab8db23..eb2133d32b6b62 100644 --- a/llvm/lib/Transforms/IPO/OpenMPOpt.cpp +++ b/llvm/lib/Transforms/IPO/OpenMPOpt.cpp @@ -549,7 +549,6 @@ struct OMPInformationCache : public InformationCache { void recollectUses() { for (int Idx = 0; Idx < RFIs.size(); ++Idx) recollectUsesForFunction(static_cast(Idx)); - OMPBuilder.Config.IsTargetDevice = isOpenMPDevice(OMPBuilder.M); } // Helper function to inherit the calling convention of the function callee.