[PATCH] D122841: [analyzer] Consider all addrspaces in null dereference check

[Balázs Benics via Phabricator via cfe-commits]

steakhal added a comment.

Couls you please provide some context about why did we have this suppression in the first place?

Other than that, I'm on board with this, but we should really investigate the historical reasons why we had this.

If it turns out to be a valid usecase, but a rather special one we could still have an analyzer config option making this change optional.

@NoQ do you have any background on this?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122841