[PATCH] D74994: [LTO] Rename legacy LTO files. NFC.

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 24 12:57:32 PST 2020


sbc100 added a comment.

In D74994#1889957 <https://reviews.llvm.org/D74994#1889957>, @steven_wu wrote:

> I don't feel like this is necessary and creating a subdirectory without creating a new library feels weird. I agree the layout is a bit weird for new comer to LTO library but it is a relatively small hurdle. I don't exactly feel like I want to pay the cost of making `git blame` a bit harder.
>
> Any other reason for changing the layout?


Just what I said in the comment.   I'm not wedded to this change.   Will drop it if others don't think it makes sense.

Out of interest do you know if there are plans to remove the legacy code one day?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74994





More information about the llvm-commits mailing list