[PATCH] D69817: Fix OrcError build with modules enabled.

Lang Hames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 5 18:11:25 PST 2019


lhames added a comment.

I think Chris is right. This isn’t an ideal breakdown of the library, but it’s a step in the right direction: OrcError should should only depend on Support (not the rest of ExecutionEngine), and ExecutionEngine should depend on OrcError.

I’ll look at this tomorrow and see if I can figure out how to structure the modules to match that layering.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69817





More information about the llvm-commits mailing list