[PATCH] D88309: [LTO][Legacy] Add API to set result type to assembly

Hubert Tong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 25 09:33:35 PDT 2020


hubert.reinterpretcast added a comment.

In D88309#2295142 <https://reviews.llvm.org/D88309#2295142>, @steven_wu wrote:

> - If this is just a temporary workaround, I would resist make this change to bake in this API that will not be used in the future.

This seems to be conflating the immediate reason with the more general value of having such an interface. The interface that @tejohnson's earlier comment mentions presumably exists because there is value in having such an interface.


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

https://reviews.llvm.org/D88309



More information about the llvm-commits mailing list