[PATCH] D72386: [ThinLTO] pass UnrollLoops/VectorizeLoop/VectorizeSLP in CGOpts down to pass builder in ltobackend

[Wei Mi via Phabricator via llvm-commits]

wmi created this revision.
wmi added a reviewer: tejohnson.
Herald added subscribers: dang, dexonsmith, steven_wu, hiraditya, inglorion, mehdi_amini, mgorny.
Herald added a project: LLVM.
wmi updated this revision to Diff 236766.

Currently UnrollLoops/VectorizeLoop/VectorizeSLP in CGOpts are not passed down to pass builder in ltobackend when new pass manager is used. This is inconsistent with the behavior when new pass manager is used and thinlto is not used. Such inconsistency causes slp vectorization pass not being enabled in ltobackend for O3 <https://reviews.llvm.org/owners/package/3/> + thinlto right now. This patch fixes that.


Repository:
  rL LLVM

https://reviews.llvm.org/D72386

Files:
  clang/test/CodeGen/thinlto-debug-pm.c


Index: clang/test/CodeGen/thinlto-debug-pm.c
===================================================================
--- clang/test/CodeGen/thinlto-debug-pm.c
+++ clang/test/CodeGen/thinlto-debug-pm.c
@@ -15,8 +15,7 @@
 
 // Test to ensure the SLP flag is passed down to the ThinLTO backend.
 // RUN: %clang_cc1 -triple x86_64-unknown-linux-gnu -emit-obj -O2 -vectorize-slp
-// -mllvm -vectorize-slp=false -mllvm -vectorize-loops=false -o %t2.o -x ir %t.o
-// -fthinlto-index=%t.thinlto.bc -fdebug-pass-manager
+// -o %t2.o -x ir %t.o -fthinlto-index=%t.thinlto.bc -fdebug-pass-manager
 // -fexperimental-new-pass-manager 2>&1 | FileCheck %s --check-prefix=O2-SLP
 // RUN: %clang_cc1 -triple x86_64-unknown-linux-gnu -emit-obj -O0 -vectorize-slp
 // -o %t2.o -x ir %t.o -fthinlto-index=%t.thinlto.bc -fdebug-pass-manager


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D72386.236766.patch
Type: text/x-patch
Size: 817 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200108/c74360fc/attachment.bin>