diff options
| author | Sam Elliott <ashe2@cs.washington.edu> | 2017-08-20 01:30:45 +0000 |
|---|---|---|
| committer | Sam Elliott <ashe2@cs.washington.edu> | 2017-08-20 01:30:45 +0000 |
| commit | b0c975369199c5f0e2c756020dff2520a9c4c2f6 (patch) | |
| tree | 8bc7e3d62676bc172f0614e2ce5a79e55f5a4138 /llvm/test/Transforms/SampleProfile/remarks.ll | |
| parent | 6adf81b1d57468c2a399083e84764abe8efba2fc (diff) | |
| download | bcm5719-llvm-b0c975369199c5f0e2c756020dff2520a9c4c2f6.tar.gz bcm5719-llvm-b0c975369199c5f0e2c756020dff2520a9c4c2f6.zip | |
Keep Optimization Remark Yaml in NewPM
Summary:
The New Pass Manager infrastructure was forgetting to keep around the optimization remark yaml file that the compiler might have been producing. This meant setting the option to '-' for stdout worked, but setting it to a filename didn't give file output (presumably it was deleted because compilation didn't explicitly keep it). This change just ensures that the file is kept if compilation succeeds.
So far I have updated one of the optimization remark output tests to add a version with the new pass manager. It is my intention for this patch to also include changes to all tests that use `-opt-remark-output=` but I wanted to get the code patch ready for review while I was making all those changes.
Fixes https://bugs.llvm.org/show_bug.cgi?id=33951
Reviewers: anemet, chandlerc
Reviewed By: anemet, chandlerc
Subscribers: javed.absar, chandlerc, fhahn, llvm-commits
Differential Revision: https://reviews.llvm.org/D36906
llvm-svn: 311271
Diffstat (limited to 'llvm/test/Transforms/SampleProfile/remarks.ll')
| -rw-r--r-- | llvm/test/Transforms/SampleProfile/remarks.ll | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/llvm/test/Transforms/SampleProfile/remarks.ll b/llvm/test/Transforms/SampleProfile/remarks.ll index 6285d69f9c7..1fc8f38751e 100644 --- a/llvm/test/Transforms/SampleProfile/remarks.ll +++ b/llvm/test/Transforms/SampleProfile/remarks.ll @@ -1,6 +1,8 @@ ; RUN: opt < %s -sample-profile -sample-profile-file=%S/Inputs/remarks.prof -S -pass-remarks=sample-profile -pass-remarks-output=%t.opt.yaml 2>&1 | FileCheck %s -; RUN: opt < %s -passes=sample-profile -sample-profile-file=%S/Inputs/remarks.prof -S -pass-remarks=sample-profile 2>&1 | FileCheck %s ; RUN: FileCheck %s -check-prefix=YAML < %t.opt.yaml +; RUN: opt < %s -passes=sample-profile -sample-profile-file=%S/Inputs/remarks.prof -S -pass-remarks=sample-profile -pass-remarks-output=%t.opt.yaml 2>&1 | FileCheck %s +; RUN: FileCheck %s -check-prefix=YAML < %t.opt.yaml + ; Original test case. ; ; 1 #include <stdlib.h> |
