steven_wu added a comment.

In D108881#2973719 <https://reviews.llvm.org/D108881#2973719>, @mnadeem wrote:

> In D108881#2973516 <https://reviews.llvm.org/D108881#2973516>, @steven_wu 
> wrote:
>
>> I will do a cleanup of `parseLTOMode` function since we don't need a 
>> `OptPos` parameter anymore. There are few minor places references `OPT_flto` 
>> or `OPT_foffload_lto` can be cleaned up too.
>
> Will you incorporate the functional changes in this patch? Or is there still 
> a need for this change?

The current change set in this review is functional change while the cleanup I 
want is not functional after the rewrite the old option as Alias. Once flto is 
the alias, there is no need to handle that in the driver and those might 
actually become source of bug in the future.

I think it would be good to do the cleanup in the same commit unless you have 
compelling reason not to.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108881

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to