[OpenMP][OMPIRBuilder] Remove redundant initialization of IsTargetDevice (#140)

skatrak · web-flow · commit 92dc51c576db · 2024-08-14T12:16:47.000+01:00
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](llvm#67000 (comment))).
diff --git 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<RuntimeFunction>(Idx));
-    OMPBuilder.Config.IsTargetDevice = isOpenMPDevice(OMPBuilder.M);
   }
 
   // Helper function to inherit the calling convention of the function callee.

Original file line number	Diff line number	Diff line change
`@@ -549,7 +549,6 @@ struct OMPInformationCache : public InformationCache {`
`549`	`549`	`void recollectUses() {`
`550`	`550`	`for (int Idx = 0; Idx < RFIs.size(); ++Idx)`
`551`	`551`	`recollectUsesForFunction(static_cast<RuntimeFunction>(Idx));`
`552`		`- OMPBuilder.Config.IsTargetDevice = isOpenMPDevice(OMPBuilder.M);`
`553`	`552`	`}`
`554`	`553`
`555`	`554`	`// Helper function to inherit the calling convention of the function callee.`