[PATCH] D48561: [Analyzer] Moved RangeConstraintManager to header. NFC.

2018-06-28 Thread Mikhail Ramalho via Phabricator via cfe-commits
mikhail.ramalho abandoned this revision. mikhail.ramalho added a comment. We won't need this patch anymore because we'll try another approach in https://reviews.llvm.org/D48565. Repository: rC Clang https://reviews.llvm.org/D48561 ___ cfe-commit

[PATCH] D48561: [Analyzer] Moved RangeConstraintManager to header. NFC.

2018-06-28 Thread Mikhail Ramalho via Phabricator via cfe-commits
mikhail.ramalho added a comment. In https://reviews.llvm.org/D48561#1146114, @george.karpenkov wrote: > After thinking about this change a bit longer, I think it does not make sense. > > Albeit poorly named, the previous design had a purpose: > `RangedConstraintManager` is a public interface, an

[PATCH] D48561: [Analyzer] Moved RangeConstraintManager to header. NFC.

2018-06-27 Thread George Karpenkov via Phabricator via cfe-commits
george.karpenkov reopened this revision. george.karpenkov added a comment. This revision is now accepted and ready to land. After thinking about this change a bit longer, I think it does not make sense. Albeit poorly named, the previous design had a purpose: `RangedConstraintManager` is a public

[PATCH] D48561: [Analyzer] Moved RangeConstraintManager to header. NFC.

2018-06-27 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC335726: [Analyzer] Moved RangeConstraintManager to header. NFC. (authored by mramalho, committed by ). Herald added a subscriber: cfe-commits. Repository: rC Clang https://reviews.llvm.org/D48561 File