<div dir="ltr">That is most likely due to the changes you've made. All tests currently pass on the bots.<div><br></div><div>- Stefan</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jan 25, 2021 at 5:27 PM Abid Malik via Openmp-dev <<a href="mailto:openmp-dev@lists.llvm.org">openmp-dev@lists.llvm.org</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"><div dir="ltr"><div class="gmail_default" style="font-family:"courier new",monospace;font-size:small">Thanks.</div><div class="gmail_default" style="font-family:"courier new",monospace;font-size:small"><br></div><div class="gmail_default" style="font-family:"courier new",monospace;font-size:small">My understanding is that all the tests available in test/Transform/OpenMP/* should pass. However, I tried and some are failing. E.g., pararllel_deletion.ll is failing. Just want to confirm if I might be doing something wrong.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jan 25, 2021 at 10:57 AM Michael Kruse <<a href="mailto:llvm@meinersbur.de" target="_blank">llvm@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 Mo., 25. Jan. 2021 um 08:11 Uhr schrieb Abid Malik via Openmp-dev<br>
<<a href="mailto:openmp-dev@lists.llvm.org" target="_blank">openmp-dev@lists.llvm.org</a>>:<br>
> 1) Why running. opt twice on some parallel region tests?<br>
> ; RUN: opt -S -attributor -openmpopt  < %s | FileCheck %s<br>
> ; RUN: opt -S -passes='attributor,cgscc(openmpopt)'  < %s | FileCheck %s<br>
<br>
The first line is testing with the legacy pass manager (at least this<br>
was the idea at some point, with EXPERIMENTAL_NEW_PASS_MANAGER=ON flag<br>
it tries to use the new pass manager even when using the legacy pass<br>
manager command line interface), the latter with the new pass manager.<br>
It is typical to have multiple run lines to run the same test input<br>
with different options.<br>
<br>
> 2) Why we are not using -O3 flag? does -openmpopt flag enough for OpenMPOpt?<br>
<br>
-O3 would add many other passes to the pass manager, but this test is<br>
meant to only test the attributor. Add unrelated passes would render<br>
the test very flaky as any change of these could cause this test to<br>
fail.<br>
<br>
Testing with -O3 would be more akin to an end-to-end test, which is<br>
what we do with the llvm-test-suite.<br>
<br>
> 3) It does not print any text or statement for debugging<br>
<br>
Tests are not supposed to check debug output. Debug output is meant to<br>
help to find an issue, i.e humans-readable, whereas regression tests<br>
only check the (machine-readable) output. To get more detailed debug<br>
output, you might want to add more LLVM_DEBUG(dbgs() << ...) which, if<br>
checked in regression tests, would cause tests to break.<br>
<br>
Unfortunately we do have many tests that also check debug output.<br>
<br>
<br>
Michael<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr"><div dir="ltr"><div>Abid M. Malik<br>******************************************************<br>"I have learned silence from the talkative, toleration from the intolerant, and kindness from the unkind"---Gibran<br>"Success is not for the chosen few, but for the few who choose" --- John Maxwell<br>"Being a good person does not depend on your religion or status in life, your race or skin color, political views or culture. IT DEPENDS ON HOW GOOD YOU TREAT OTHERS"--- Abid<br>"The Universe is talking to us, and the language of the Universe is mathematics."----Abid<br> </div></div></div>
_______________________________________________<br>
Openmp-dev mailing list<br>
<a href="mailto:Openmp-dev@lists.llvm.org" target="_blank">Openmp-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/openmp-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/openmp-dev</a><br>
</blockquote></div>