[PATCH] D108881: [clang][driver] Honor the last -flto(=.*)? argument

Mon Aug 30 18:34:12 PDT 2021

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