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

Juneyoung Lee via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 12 16:32:36 PST 2021


aqjune added a comment.

In D92015#2493214 <https://reviews.llvm.org/D92015#2493214>, @nikic wrote:

> 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.

Okay. Since it would be desirable for the reported performance bug to be closed before branching, I'll land this patch.
I'll leave a link to this patch at comments as well.


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