[PATCH] D143867: [analyzer] Fix SARIF column location assertion crash

[Scarlet via Phabricator via cfe-commits]

Scarlet1ssimo added a comment.

In D143867#4121835 <https://reviews.llvm.org/D143867#4121835>, @steakhal wrote:

> Looks sensible to me.
> Do you have a test for triggering the previous assertion?

I do have prepared a test case like:

  int testA(void) { return 0/0; }
  // RUN: %clang_analyze_cc1 -analyzer-checker=core %s -analyzer-output=sarif -o - 

Notice you must put the buggy code at the very first line to trigger the assertion.

Do you think it's necessary to put this test case into the test suite?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143867