[PATCH] D145644: [memprof] Add scripts to automate testdata regeneration.

2023-03-09 Thread Snehasish Kumar via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGe99b5ad38381: [memprof] Add scripts to automate testdata regeneration. (authored by snehasish). Repository: rG LLVM Github Monorepo CHANGES SINCE

[PATCH] D145644: [memprof] Add scripts to automate testdata regeneration.

2023-03-09 Thread Snehasish Kumar via Phabricator via cfe-commits
snehasish added a comment. I'll go ahead and push this. @dblaikie Let me know if you have any comments and I'll follow up separately. Thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145644/new/ https://reviews.llvm.org/D145644 _

[PATCH] D145644: [memprof] Add scripts to automate testdata regeneration.

2023-03-09 Thread Teresa Johnson via Phabricator via cfe-commits
tejohnson accepted this revision. tejohnson added a comment. This revision is now accepted and ready to land. lgtm Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145644/new/ https://reviews.llvm.org/D145644 _

[PATCH] D145644: [memprof] Add scripts to automate testdata regeneration.

2023-03-09 Thread Snehasish Kumar via Phabricator via cfe-commits
snehasish marked an inline comment as done. snehasish added a comment. In D145644#4182239 , @tejohnson wrote: > Generally lgtm, but why did the raw profiles change size from what is > currently committed? The reduction in size of the raw profiles is due

[PATCH] D145644: [memprof] Add scripts to automate testdata regeneration.

2023-03-09 Thread Snehasish Kumar via Phabricator via cfe-commits
snehasish updated this revision to Diff 503859. snehasish added a comment. Update the script in test/tool/llvm-profdata/Inputs. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145644/new/ https://reviews.llvm.org/D145644 Files: clang/test/CodeGen/

[PATCH] D145644: [memprof] Add scripts to automate testdata regeneration.

2023-03-09 Thread Teresa Johnson via Phabricator via cfe-commits
tejohnson added a comment. Generally lgtm, but why did the raw profiles change size from what is currently committed? Comment at: llvm/test/Transforms/PGOProfile/Inputs/update_memprof_inputs.sh:63 + delete[] f; + + // Loop ensures the two calls to recurse have stack contexts

[PATCH] D145644: [memprof] Add scripts to automate testdata regeneration.

2023-03-09 Thread Snehasish Kumar via Phabricator via cfe-commits
snehasish added inline comments. Comment at: llvm/test/Transforms/PGOProfile/Inputs/update_memprof_inputs.sh:63 + delete[] f; + + // Loop ensures the two calls to recurse have stack contexts that only differ @tejohnson Turns out the LLVM IR matching issue was d

[PATCH] D145644: [memprof] Add scripts to automate testdata regeneration.

2023-03-09 Thread Snehasish Kumar via Phabricator via cfe-commits
snehasish updated this revision to Diff 503850. snehasish added a comment. Update the script for Transforms/PGOProfile/memprof.ll. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145644/new/ https://reviews.llvm.org/D145644 Files: clang/test/CodeG

[PATCH] D145644: [memprof] Add scripts to automate testdata regeneration.

2023-03-08 Thread Snehasish Kumar via Phabricator via cfe-commits
snehasish created this revision. snehasish added reviewers: tejohnson, dblaikie. Herald added a subscriber: wenlei. Herald added a project: All. snehasish requested review of this revision. Herald added projects: clang, LLVM. Herald added subscribers: llvm-commits, cfe-commits. The memprof profile