[PATCH] D90278: [ThinLTO] Fix .llvmcmd emission

Mircea Trofin via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 28 11:14:50 PDT 2020


mtrofin added a comment.

In D90278#2359674 <https://reviews.llvm.org/D90278#2359674>, @tejohnson wrote:

> LGTM but is there a functional reason why CmdArgs was changed to be passed by reference? If just generic cleanup might be better to split up that into a separate commit.

It avoids having to also check if that thing's empty. I can do that first as a NFC and re-sync.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90278



More information about the cfe-commits mailing list