[PATCH] D87336: [NFC][ThinLTO] EmbedBitcodeSection doesn't need the Config

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 8 16:51:09 PDT 2020

Previous message: [llvm] efc17c4 - [NFC] Fixing a gcc compiler warning.
Next message: [PATCH] D87336: [NFC][ThinLTO] EmbedBitcodeSection doesn't need the Config
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

tejohnson added a comment.

Why not just make this one an NFC change to remove the unused Conf parameter. Then pass in CmdLineOpts in the patch where its use is added?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87336

Previous message: [llvm] efc17c4 - [NFC] Fixing a gcc compiler warning.
Next message: [PATCH] D87336: [NFC][ThinLTO] EmbedBitcodeSection doesn't need the Config
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list