[llvm] f39c385 - [MLGO] Fix tests post 1a2e77c

Aiden Grossman via llvm-commits llvm-commits at lists.llvm.org
Sun Oct 29 01:41:17 PDT 2023


Author: Aiden Grossman
Date: 2023-10-29T08:41:11Z
New Revision: f39c38584eb762702a651e87e63162c9bc4842a3

URL: https://github.com/llvm/llvm-project/commit/f39c38584eb762702a651e87e63162c9bc4842a3
DIFF: https://github.com/llvm/llvm-project/commit/f39c38584eb762702a651e87e63162c9bc4842a3.diff

LOG: [MLGO] Fix tests post 1a2e77c

This patch switched the default value of the mandatory-inlining-first
flag from true to false. This broke one of the MLGO tests that relied on
the default value of this flag. This patch explicitly sets the value to
fix the test and avoid future breakages.

Added: 
    

Modified: 
    llvm/test/Transforms/Inline/ML/enable-inline-advisor-printing-ml.ll

Removed: 
    


################################################################################
diff  --git a/llvm/test/Transforms/Inline/ML/enable-inline-advisor-printing-ml.ll b/llvm/test/Transforms/Inline/ML/enable-inline-advisor-printing-ml.ll
index 84e51db8d94fddf..376548b3f06feef 100644
--- a/llvm/test/Transforms/Inline/ML/enable-inline-advisor-printing-ml.ll
+++ b/llvm/test/Transforms/Inline/ML/enable-inline-advisor-printing-ml.ll
@@ -1,7 +1,7 @@
 ; REQUIRES: llvm_inliner_model_autogenerated
 
 ; RUN: opt -enable-ml-inliner=release -passes=scc-oz-module-inliner \
-; RUN:     -keep-inline-advisor-for-printing \
+; RUN:     -keep-inline-advisor-for-printing -mandatory-inlining-first=1 \
 ; RUN:     -enable-scc-inline-advisor-printing -S < %s 2>&1 | FileCheck %s
 
 ; RUN: opt -enable-ml-inliner=release -passes=scc-oz-module-inliner \


        


More information about the llvm-commits mailing list