[PATCH] D105835: [Driver] Let -fno-integrated-as -gdwarf-5 use -fdwarf-directory-asm

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 12 15:02:03 PDT 2021


MaskRay added a comment.

In D105835#2872393 <https://reviews.llvm.org/D105835#2872393>, @osandov wrote:

> I tested my reproducer and it also fixes it, thanks. Should it be an error to specify `-fno-dwarf-directory-asm` together with `-gdwarf-5`, since that produces incorrect results?

I thought about this, but did not add the code because I think the option `-fno-dwarf-directory-asm` is so obscure that probably nobody will use...
Not sure why it became a driver option...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105835



More information about the cfe-commits mailing list