[PATCH] D64784: Skip zero column for inline sites

Wenlei He via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 15 21:03:03 PDT 2019


wenlei added a comment.

In D64784#1586900 <https://reviews.llvm.org/D64784#1586900>, @dblaikie wrote:

> Test coverage? (you could add another inlined call to the test case you added for the feature - but just leave the inlined-at location with a zero column for that particular call - so you can still test both positive/negatiev in the one file)


Thanks! done.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D64784





More information about the llvm-commits mailing list