[PATCH] D65532: [LTO][Legacy] Update TargetOption after parsing debug options

[Teresa Johnson via Phabricator via llvm-commits]

tejohnson added a comment.

I just discovered that there was a nearly identical patch to do this a few years ago, which generated a bunch of discussion and it seems that there was an agreement not to do this (relating to not exposing internal options through the C API, but I haven't parsed it in detail yet). See https://reviews.llvm.org/D19015

However, note that only about the first half of the discussion is on Phabricator, the other half was via email. Search for D19015 <https://reviews.llvm.org/D19015> here: http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20160411/thread.html

Adding @mehdi_amini and @probinson here since they were involved in the earlier patch discussion.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65532