[PATCH] D67205: [SimplifyCFG] Don't SimplifyBranchOnICmpChain with ExtraCase

[Vitaly Buka via Phabricator via llvm-commits]

vitalybuka added a comment.

In D67205#1659684 <https://reviews.llvm.org/D67205#1659684>, @efriedma wrote:

> You description of the issue uses "||" in the pseudo-code; I assume you actually mean a non-short-circuiting "|"?


yes, updated description

> Is there a general description somewhere of what restrictions msan places on IR transforms vs. normal IR semantics?  It's not obvious to me that this transform, specifically, is incorrect, as opposed to whatever transform is producing this IR.

I don't have one. @eugenis ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67205