[PATCH] D54653: Remove branch from CreateAlignmentAssumption

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri accepted this revision.
lebedev.ri added a comment.

Ah, ok.
Since we no longer do "ispositive" check, we will no longer errneously ignore `unsigned(INT_MIN)` alignment, so this should still address my initial concern.
And extra UB is super awesome, and will surely be caught by the sanitizer (once the [[ D54590 <https://reviews.llvm.org/D54590> | compiler-rt part ]] is reviewed, that is.) :)


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

https://reviews.llvm.org/D54653