[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