[llvm] [LTO] Compare std::optional<ImportKind> directly with ImportKind (NFC) (PR #105561)
Kazu Hirata via llvm-commits
llvm-commits at lists.llvm.org
Wed Aug 21 11:00:46 PDT 2024
https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/105561
Note that:
Opt == Val if and only (Opt && *Opt == Val)
where:
std::optional<T> Opt;
T Val;
>From 8d4d0dc9d98cde7a46a8d88b30d6690b6eaa11a0 Mon Sep 17 00:00:00 2001
From: Kazu Hirata <kazu at google.com>
Date: Wed, 21 Aug 2024 10:46:16 -0700
Subject: [PATCH] [LTO] Compare std::optional<ImportKind> directly with
ImportKind (NFC)
Note that:
Opt == Val if and only (Opt && *Opt == Val)
where:
std::optional<T> Opt;
T Val;
---
llvm/lib/Transforms/IPO/FunctionImport.cpp | 15 +++------------
1 file changed, 3 insertions(+), 12 deletions(-)
diff --git a/llvm/lib/Transforms/IPO/FunctionImport.cpp b/llvm/lib/Transforms/IPO/FunctionImport.cpp
index 6ae89a49b6b9a3..92371720e0eceb 100644
--- a/llvm/lib/Transforms/IPO/FunctionImport.cpp
+++ b/llvm/lib/Transforms/IPO/FunctionImport.cpp
@@ -1814,10 +1814,7 @@ Expected<bool> FunctionImporter::importFunctions(
continue;
auto GUID = F.getGUID();
auto MaybeImportType = getImportType(ImportGUIDs, GUID);
-
- bool ImportDefinition =
- (MaybeImportType &&
- (*MaybeImportType == GlobalValueSummary::Definition));
+ bool ImportDefinition = MaybeImportType == GlobalValueSummary::Definition;
LLVM_DEBUG(dbgs() << (MaybeImportType ? "Is" : "Not")
<< " importing function"
@@ -1853,10 +1850,7 @@ Expected<bool> FunctionImporter::importFunctions(
continue;
auto GUID = GV.getGUID();
auto MaybeImportType = getImportType(ImportGUIDs, GUID);
-
- bool ImportDefinition =
- (MaybeImportType &&
- (*MaybeImportType == GlobalValueSummary::Definition));
+ bool ImportDefinition = MaybeImportType == GlobalValueSummary::Definition;
LLVM_DEBUG(dbgs() << (MaybeImportType ? "Is" : "Not")
<< " importing global"
@@ -1876,10 +1870,7 @@ Expected<bool> FunctionImporter::importFunctions(
continue;
auto GUID = GA.getGUID();
auto MaybeImportType = getImportType(ImportGUIDs, GUID);
-
- bool ImportDefinition =
- (MaybeImportType &&
- (*MaybeImportType == GlobalValueSummary::Definition));
+ bool ImportDefinition = MaybeImportType == GlobalValueSummary::Definition;
LLVM_DEBUG(dbgs() << (MaybeImportType ? "Is" : "Not")
<< " importing alias"
More information about the llvm-commits
mailing list