<div dir="ltr"><div>Yes we should make sure tests pass with the legacy PM as the default, treating the bot like any other bot.</div>I've fixed the two func-attrs failures.<div>The two OpenMP failures look like real issues under the legacy PM that need to be investigated.</div><div><br></div><div><font face="monospace">diff --git a/llvm/test/Transforms/OpenMP/hide_mem_transfer_latency.ll b/llvm/test/Transforms/OpenMP/hide_mem_transfer_latency.ll<br>index abc0d8769c24..fd2e499d9a28 100644<br>--- a/llvm/test/Transforms/OpenMP/hide_mem_transfer_latency.ll<br>+++ b/llvm/test/Transforms/OpenMP/hide_mem_transfer_latency.ll<br>@@ -1,5 +1,5 @@<br> ; NOTE: Assertions have been autogenerated by utils/update_test_checks.py UTC_ARGS: -p --function-signature --scrub-attributes<br>-; RUN: opt -S -openmp-opt-cgscc -aa-pipeline=basic-aa -openmp-hide-memory-transfer-latency < %s | FileCheck %s<br>+; RUN: opt -S -openmp-opt-cgscc -enable-new-pm=0 -openmp-hide-memory-transfer-latency < %s | FileCheck %s<br> ; RUN: opt -S -passes=openmp-opt-cgscc -aa-pipeline=basic-aa -openmp-hide-memory-transfer-latency < %s | FileCheck %s<br> target datalayout = "e-m:e-p270:32:32-p271:32:32-p272:64:64-i64:64-f80:128-n8:16:32:64-S128"<br></font></div><div><br></div><div>to repro.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Apr 27, 2021 at 12:40 PM Michael Kruse <<a href="mailto:llvmdev@meinersbur.de">llvmdev@meinersbur.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Am Di., 27. Apr. 2021 um 10:59 Uhr schrieb Arthur Eubanks via llvm-dev<br>
<<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>>:<br>
> We'd still have to have support in opt for the legacy PM to run IR passes that are part of the codegen pipeline. But we could still remove the -enable-new-pm flag after moving tests for passes in the optimization pipeline to use -passes.<br>
<br>
It might be worth noting that the regression tests are currently<br>
failing with ENABLE_EXPERIMENTAL_NEW_PASS_MANAGER=OFF. If it is still<br>
supported till the next release, they should be fixed.<br>
<br>
See <a href="http://meinersbur.de:8011/#/builders/148" rel="noreferrer" target="_blank">http://meinersbur.de:8011/#/builders/148</a><br>
<br>
Michael<br>
</blockquote></div>