[PATCH] D94882: [MC] Upgrade DWARF version to 5 upon .file 0

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jan 17 10:50:32 PST 2021


MaskRay added a comment.
Herald added a subscriber: ormris.

Theoretically the diagnostic can be retained: we need another variable beside `MCContext::DwarfVersion` or making `MCContext::DwarfVersion` `Optional<int>`; and carefully doing similar thing on Clang cc1as side.

The diagnostic may not be worth the additional code complexity, though.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94882



More information about the llvm-commits mailing list