[llvm] [MLGO] Remove -tfutils-use-simplelogger flag (PR #72492)

Aiden Grossman via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 16 00:31:21 PST 2023


https://github.com/boomanaiden154 created https://github.com/llvm/llvm-project/pull/72492

This flag was redundant and the value was not used anywhere, so it should be removed.

>From bc34bfb0f896cae2f7021a72a916e904ba30ab0b Mon Sep 17 00:00:00 2001
From: Aiden Grossman <agrossman154 at yahoo.com>
Date: Thu, 16 Nov 2023 00:28:19 -0800
Subject: [PATCH] [MLGO] Remove -tfutils-use-simplelogger flag

This flag was redundant and the value was not used anywhere, so it
should be removed.
---
 llvm/lib/Analysis/TrainingLogger.cpp | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/llvm/lib/Analysis/TrainingLogger.cpp b/llvm/lib/Analysis/TrainingLogger.cpp
index e236890aa2bcc22..344ca92e18b5191 100644
--- a/llvm/lib/Analysis/TrainingLogger.cpp
+++ b/llvm/lib/Analysis/TrainingLogger.cpp
@@ -27,11 +27,6 @@
 
 using namespace llvm;
 
-// FIXME(mtrofin): remove the flag altogether
-static cl::opt<bool>
-    UseSimpleLogger("tfutils-use-simplelogger", cl::init(true), cl::Hidden,
-                    cl::desc("Output simple (non-protobuf) log."));
-
 void Logger::writeHeader(std::optional<TensorSpec> AdviceSpec) {
   json::OStream JOS(*OS);
   JOS.object([&]() {



More information about the llvm-commits mailing list