[PATCH] D44788: Add an option to support debug fission on implicit ThinLTO.

Yunlian Jiang via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 20 09:58:28 PDT 2018


yunlian added inline comments.


================
Comment at: lib/Driver/ToolChains/CommonArgs.cpp:428
+
+  if (Args.hasArg(options::OPT_gsplit_dwarf)) {
+    if (!Args.getLastArg(options::OPT_gsplit_dwarf_EQ)) {
----------------
pcc wrote:
> If you make this `else if (Args.hasArg(options::OPT_gsplit_dwarf)) {` you wouldn't need the if on line 429.
Will do.


https://reviews.llvm.org/D44788





More information about the cfe-commits mailing list