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

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 12 11:45:31 PST 2018


dblaikie added a comment.

Looks like the original functionality was added here in r109651 without any test case... so it's hard to say that because your change causes no regression that it's correct, because there's a lack of test coverage (no test tests for this message about "file number already allocated").

So probably need to have some discussion about when/where/if this should fire.


Repository:
  rL LLVM

https://reviews.llvm.org/D54348





More information about the llvm-commits mailing list