[PATCH] D92611: [LTO][Legacy] Decouple option parsing from LTOCodeGenerator

Steven Wu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 30 11:09:51 PDT 2021


steven_wu added a comment.

In D92611#2659079 <https://reviews.llvm.org/D92611#2659079>, @w2yehia wrote:

> @steven_wu thanks for your review. Is this good to be merged then?

Can you just revert to use `parsedOptions` as boolean? The state you used right now doesn't add anything. You can just leave `lto_set_debug_options` not guarded right?


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

https://reviews.llvm.org/D92611



More information about the llvm-commits mailing list