[PATCH] D54348: Permit multiple .file directives with -g

Paul Robinson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 17 10:21:48 PDT 2019


probinson added a comment.

As I mentioned in PR38449, I plan to look at this starting this week.  Even benign situations such as

  foo:
  .file 1 "a.c"

are tripping over the assertion.  I think the correct tactic is not to remove the places that are doing the checks, but make those places do a better job of tidying up anything that had been done in response to the command-line `-g` in favor of allowing the embedded directives to DTRT.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D54348





More information about the llvm-commits mailing list