[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