[PATCH] D23315: [ELF] - Let linkerscript to use own outputsections factory.

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 11 14:51:09 PDT 2016


ruiu added a comment.

I do not see a benefit of doing this with a factory class. Actually I'm starting to regret to introduce the factory class to do this relatively simple things. We are doing too much object-oriented dance here. I believe this can be done just by a simple function.


https://reviews.llvm.org/D23315





More information about the llvm-commits mailing list