[PATCH] D20682: Expose target-specific pre-linking passes in TargetMachine

Anastasia Stulova via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 14 08:50:22 PDT 2016


Anastasia added a comment.

In http://reviews.llvm.org/D20682#457117, @chandlerc wrote:

> I fundamentally do not believe that the TargetMachine should be involved in fixing language semantics issues with a "pre linking" pass at the LLVM level. Why? Because there is nothing "target" about this.
>
> There needs to be a fundamentally more principled way of handling this at the language and frontend level IMO.


I personally don't think adding more target specific details to frontend is a good idea from the point of view of modular design.

It would be nice to understand why you think that this is not target enough for this purpose.


http://reviews.llvm.org/D20682





More information about the llvm-commits mailing list