[PATCH] D139223: [ORC] Move LinkGraphMaterializationUnit for testing (NFC)

Keith Smiley via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 2 12:52:42 PST 2022


keith added a comment.

this isn't exactly what was discussed in https://reviews.llvm.org/D130221 but it seems like making `LinkGraphMaterializationUnit` accessible is nicer for unit tests where we only care to test its behavior vs the whole system. lmk what you think. Locally I also have a patch to use `MachOPlatform::isInitializerSection` instead of the local version here but I wanted to prove the tests pass before and after so I didn't include it yet.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D139223/new/

https://reviews.llvm.org/D139223



More information about the llvm-commits mailing list