[PATCH] D124347: [BOLT] RFC: Add golang support
Amir Ayupov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed May 4 13:45:17 PDT 2022
Amir added a comment.
Vladislav, thank you for adding this support! It's a massive change, so let's make it easier to review. Can you please split this diff up into small isolated chunks?
I would suggest having separate diffs for:
1. Core interface changes/additions
2. Instrumentation changes
3. MCPlusBuilder changes
4. Passes changes
5. Base Golang passes/functions/data structures
6. Interaction between Golang stuff and existing passes
7. Documentation changes
An example of such a stack would be D124900 <https://reviews.llvm.org/D124900>.
It would be much easier to integrate the work into the codebase with bite-sized diffs.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D124347/new/
https://reviews.llvm.org/D124347
More information about the llvm-commits
mailing list