[llvm] [MLGO] Remove -tfutils-use-simplelogger flag (PR #72492)
    via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Thu Nov 16 00:31:54 PST 2023
    
    
  
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-llvm-analysis
Author: Aiden Grossman (boomanaiden154)
<details>
<summary>Changes</summary>
This flag was redundant and the value was not used anywhere, so it should be removed.
---
Full diff: https://github.com/llvm/llvm-project/pull/72492.diff
1 Files Affected:
- (modified) llvm/lib/Analysis/TrainingLogger.cpp (-5) 
``````````diff
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([&]() {
``````````
</details>
https://github.com/llvm/llvm-project/pull/72492
    
    
More information about the llvm-commits
mailing list