[PATCH] D88019: [analyzer][solver] Fix issue with symbol non-equality tracking

Valeriy Savchenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 21 05:39:29 PDT 2020


vsavchenko added a comment.

I came up with exactly the same fix! Great job!
I just wanted to refactor it and not having

  if (New.isEmpty())
    // this is infeasible assumption
    return nullptr;
  
  ProgramStateRef NewState = setConstraint(St, Sym, New);
  return trackNE(NewState, Sym, Int, Adjustment);

repeated in different places


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88019



More information about the cfe-commits mailing list