[Backport to llvm_release_170] Fix potential constructor ambiguity in BuiltinCallMutator::ValueTypePair (#3271)

github-actions[bot] · jaladreips · web-flow · commit 005c78a2adcd · 2025-07-22T14:54:56.000+02:00
Backport of PR #3268 into `llvm_release_170`. All commits applied cleanly. Co-authored-by: Jakub Jakacki <jakub.jakacki@intel.com>
diff --git a/lib/SPIRV/SPIRVBuiltinHelper.h b/lib/SPIRV/SPIRVBuiltinHelper.h
@@ -139,7 +139,6 @@ class BuiltinCallMutator {
              "Must specify a pointer element type if value is a pointer.");
     }
     ValueTypePair(std::pair<llvm::Value *, llvm::Type *> P) : pair(P) {}
-    ValueTypePair(llvm::Value *V, llvm::Type *T) : pair(V, T) {}
     ValueTypePair() = delete;
     using pair::pair;
   };

Original file line number	Diff line number	Diff line change
`@@ -139,7 +139,6 @@ class BuiltinCallMutator {`
`139`	`139`	`"Must specify a pointer element type if value is a pointer.");`
`140`	`140`	`}`
`141`	`141`	`ValueTypePair(std::pair<llvm::Value , llvm::Type > P) : pair(P) {}`
`142`		`- ValueTypePair(llvm::Value V, llvm::Type T) : pair(V, T) {}`
`143`	`142`	`ValueTypePair() = delete;`
`144`	`143`	`using pair::pair;`
`145`	`144`	`};`