[PATCH] D94381: [test] Add Clang side tests for -fdebug-info-for-profiling

2021-01-13 Thread Fangrui Song via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG74a42aedfe14: [test] Add Clang side tests for -fdebug-info-for-profiling (authored by MaskRay). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94381/new/ htt

[PATCH] D94381: [test] Add Clang side tests for -fdebug-info-for-profiling

2021-01-13 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added a comment. Verified with @wmi offline this is good. Committing... Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94381/new/ https://reviews.llvm.org/D94381 ___ cfe-commits mailing list cfe-c

[PATCH] D94381: [test] Add Clang side tests for -fdebug-info-for-profiling

2021-01-12 Thread David Blaikie via Phabricator via cfe-commits
dblaikie accepted this revision. dblaikie added a comment. This revision is now accepted and ready to land. Sounds good Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94381/new/ https://reviews.llvm.org/D94381 __

[PATCH] D94381: [test] Add Clang side tests for -fdebug-info-for-profiling

2021-01-12 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added inline comments. Comment at: clang/test/CodeGenCXX/fdebug-info-for-profiling.cpp:14-16 +// RUN: echo > %t.proftext +// RUN: llvm-profdata merge %t.proftext -o %t.profdata +// RUN: %clang_cc1 -emit-llvm -fno-legacy-pass-manager -fdebug-pass-manager -O1 -fprofile-ins

[PATCH] D94381: [test] Add Clang side tests for -fdebug-info-for-profiling

2021-01-12 Thread David Blaikie via Phabricator via cfe-commits
dblaikie added inline comments. Comment at: clang/test/CodeGenCXX/fdebug-info-for-profiling.cpp:14-16 +// RUN: echo > %t.proftext +// RUN: llvm-profdata merge %t.proftext -o %t.profdata +// RUN: %clang_cc1 -emit-llvm -fno-legacy-pass-manager -fdebug-pass-manager -O1 -fprofile-in

[PATCH] D94381: [test] Add Clang side tests for -fdebug-info-for-profiling

2021-01-10 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay created this revision. MaskRay added reviewers: dblaikie, davidxl, wmi. MaskRay requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. There is currently a driver test but no test for its effect on linkageName & pass pipeline. Repositor