[flang-commits] [PATCH] D130628: [zorg] Run test-suite with flang-new

[Andrzej Warzynski via Phabricator via flang-commits]

awarzynski added a comment.

In D130628#3682115 <https://reviews.llvm.org/D130628#3682115>, @rovka wrote:

> @DavidSpickett You're right, it's probably better to put the list in the test-suite. I'll update accordingly.
>
> In D130628#3682032 <https://reviews.llvm.org/D130628#3682032>, @DavidSpickett wrote:
>
>> Also why exactly is NO_STOP_MESSAGE needed? I guess fortran tests do some stdout comparison and this would otherwise print a line that gets in the way.
>
> Yep, that's exactly it :) This is documented here <https://flang.llvm.org/docs/FortranLLVMTestSuite.html#:~:text=to%20set%20the-,NO_STOP_MESSAGE,-environment%20variable%20to>.

Perhaps worth adding a comment somewhere so that the rationale for this is clearer (either in the code or in the commit message).



================
Comment at: zorg/buildbot/builders/ClangBuilder.py:512
+                            '-flang-experimental-exec -lpgmath ' +
+                            '-Xlinker -rpath %(prop:builddir)s/'+compiler_path+'/lib'),
                         util.Interpolate(
----------------
What's this flag for? libpgmath?


Repository:
  rZORG LLVM Github Zorg

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D130628/new/

https://reviews.llvm.org/D130628