[llvm][TextAPI] Ignore deployment version in target comparator

cyndyishida · cyndyishida · commit 5b414cb6d5a6 · 2023-08-06T19:41:20.000-07:00
(cherry picked from commit d4c7802)
diff --git a/llvm/include/llvm/TextAPI/Target.h b/llvm/include/llvm/TextAPI/Target.h
@@ -54,8 +54,8 @@ inline bool operator!=(const Target &LHS, const Target &RHS) {
 }
 
 inline bool operator<(const Target &LHS, const Target &RHS) {
-  return std::tie(LHS.Arch, LHS.Platform, LHS.MinDeployment) <
-         std::tie(RHS.Arch, RHS.Platform, RHS.MinDeployment);
+  // In most cases the deployment version is not useful to compare.
+  return std::tie(LHS.Arch, LHS.Platform) < std::tie(RHS.Arch, RHS.Platform);
 }
 
 inline bool operator==(const Target &LHS, const Architecture &RHS) {

Original file line number	Diff line number	Diff line change
`@@ -54,8 +54,8 @@ inline bool operator!=(const Target &LHS, const Target &RHS) {`
`54`	`54`	`}`
`55`	`55`
`56`	`56`	`inline bool operator<(const Target &LHS, const Target &RHS) {`
`57`		`- return std::tie(LHS.Arch, LHS.Platform, LHS.MinDeployment) <`
`58`		`- std::tie(RHS.Arch, RHS.Platform, RHS.MinDeployment);`
	`57`	`+ // In most cases the deployment version is not useful to compare.`
	`58`	`+ return std::tie(LHS.Arch, LHS.Platform) < std::tie(RHS.Arch, RHS.Platform);`
`59`	`59`	`}`
`60`	`60`
`61`	`61`	`inline bool operator==(const Target &LHS, const Architecture &RHS) {`