[PATCH] D123678: [polly] migrate -polly-show to the new pass manager

[Michael Kruse via Phabricator via llvm-commits]

Meinersbur added a comment.

I could only observe a single failure with my own buildbot was assuming it was something else.

https://lab.llvm.org/buildbot/#/builders/10/builds/15406

  ******************** TEST 'Polly :: ScopDetect/dot-scops-npm.ll' FAILED ********************
  Script:
  --
  : 'RUN: at line 1';   /home/worker/buildbot-workers/polly-x86_64-gce1/rundir/llvm.obj/bin/opt  -polly-process-unprofitable  -polly-remarks-minimal  -polly-use-llvm-names  -polly-import-jscop-dir=/home/worker/src/llvm-project/polly/test/ScopDetect  -polly-codegen-verify  "-passes=polly-scop-printer" -disable-output < /home/worker/src/llvm-project/polly/test/ScopDetect/dot-scops-npm.ll
  : 'RUN: at line 2';   /home/worker/buildbot-workers/polly-x86_64-gce1/rundir/llvm.obj/bin/FileCheck /home/worker/src/llvm-project/polly/test/ScopDetect/dot-scops-npm.ll -input-file=scops.func.dot
  --
  Exit Code: 2
  Command Output (stderr):
  --
  Writing 'scops.func.dot'...
  FileCheck error: 'scops.func.dot' is empty.
  FileCheck command line:  /home/worker/buildbot-workers/polly-x86_64-gce1/rundir/llvm.obj/bin/FileCheck /home/worker/src/llvm-project/polly/test/ScopDetect/dot-scops-npm.ll -input-file=scops.func.dot
  --
  ********************

The test is suspicious in that it does not use `%t` for its temporary file, it could potentially clash with another test using the same file concurrently. I suspect it is `ScopDetect/dot-scops-npm.ll`. Don't we have a means to specify which file to write?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123678