[PATCH] D85176: [Coverage] Enable emitting gap area between macros

Zequan Wu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 26 17:04:52 PDT 2020


zequanwu reopened this revision.
zequanwu added a comment.
This revision is now accepted and ready to land.

Here is a repro of crash caused by this change.

  int k, l;
  #define m(e) e##e
  void p() {
    int kk,ll;
    if (k)
      m(k);
    else
      l = m(l);
  }

`SM.getExpansLoc(AfterLoc)` for `m(k)` gives location pointing to `m(l)`, which caused the crash.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85176



More information about the cfe-commits mailing list