[PATCH] D92015: [DAGCombiner] Fold BRCOND(FREEZE(COND)) to BRCOND(COND)

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 12 09:26:59 PST 2021


nikic added a comment.

Yes, I think this is fine to land as-is. Note that SDAG always works on a single basic block, so I don't think there is any value in having "branch on undef is UB" semantics, as we can't infer information from branch conditions anyway.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92015



More information about the llvm-commits mailing list