[PATCH] D101462: Make it possible for targets to define their own MCObjectFileInfo

Philipp Krones via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 30 02:37:43 PDT 2021


flip1995 added a comment.

> The refactoring adding `Triple` to `MCContext::MCContext` [...] should be separate.

In order to make the `MCContext` construction independent from the `MCObjectFileInfo`, passing the `Triple` to the `MCContext` is necessary anyway. Moving it completely to the `MCContext` was just the logical next step for me. It also simplifies calls across the code base since it removes an indirection over `MOFI` when accessing the triple.

I'll update this on Monday 👍


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101462



More information about the llvm-commits mailing list