⚙ D92015 [DAGCombiner] Fold BRCOND(FREEZE(COND)) to BRCOND(COND) (original) (raw)

This is an archive of the discontinued LLVM Phabricator instance.

Table of Contentst-llvm/-include/llvm/CodeGen/-llvm/-CodeGen/-ISDOpcodes.h-lib/CodeGen/SelectionDAG/-CodeGen/-SelectionDAG/2/2DAGCombiner.cpp-test/CodeGen/X86/-CodeGen/-X86/-select-prof-codegen.llHide PanelfKeyboard Reference? Differential D92015 Authored by aqjune on Nov 24 2020, 2:11 AM.Download Raw DiffReviewers craig.topperRKSimonefriedmaspatelxbolva00nlopes Commits rG25eb7b08ba77: [DAGCombiner] Fold BRCOND(FREEZE(COND)) to BRCOND(COND) Repository rG LLVM Github Monorepo Event Timelineaqjune created this revision.Herald added a project: Restricted Project. Herald added subscribers: llvm-commits, ecnelises, steven.zhang and 2 others. aqjune requested review of this revision.aqjune edited the summary of this revision. (Show Details)Harbormaster completed remote builds in B79913: Diff 307288.aqjune added a reviewer: xbolva00.RKSimon added a reviewer: nlopes.Comment ActionsThis revision is now accepted and ready to land.aqjune mentioned this in rGda1bd953faaf: Precommit a test for D92015 (NFC).Comment ActionsComment ActionsHarbormaster completed remote builds in B83802: Diff 314216.nikic added a subscriber: nikic.nikic added inline comments.llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp 14556Comment Actionsaqjune marked an inline comment as done.aqjune added inline comments.llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp 14556aqjune marked an inline comment as done.Harbormaster completed remote builds in B83828: Diff 314262.Comment ActionsComment ActionsComment ActionsComment ActionsThis revision was landed with ongoing or failed builds.Closed by commit rG25eb7b08ba77: [DAGCombiner] Fold BRCOND(FREEZE(COND)) to BRCOND(COND) (authored by aqjune). This revision was automatically updated to reflect the committed changes.aqjune added a commit: rG25eb7b08ba77: [DAGCombiner] Fold BRCOND(FREEZE(COND)) to BRCOND(COND).Harbormaster completed remote builds in B84948: Diff 316276.aqjune mentioned this in D105344: [DAGCombiner] Fold SETCC(FREEZE(x),const) to FREEZE(SETCC(x,const)) if SETCC is used by BRCOND.FilesHistoryCommitsSimilarPathSizellvm/include/llvm/CodeGen/ISDOpcodes.h9 lineslib/CodeGen/SelectionDAG/DAGCombiner.cpp7 linestest/CodeGen/X86/select-prof-codegen.ll6 linesDiffIDBaseDescriptionCreatedLintUnitBaseBaseDiff 130728882c22f1Nov 24 2020, 2:11 AM★★Diff 2314216da1bd95Rebase, precommit the testJan 1 2021, 5:44 AM★★Diff 3314262da1bd95Add one use checkJan 2 2021, 3:35 PM★★Diff 431627676643c4Leave a comment about the issueJan 12 2021, 4:57 PM★★Diff 531627776643c4rG25eb7b08ba77a0b7c9c938490444bb8b5121233cJan 12 2021, 4:36 PM★★D152928 [RFC][DAG] Initially add nodes in the worklist in topological order.·Reviewers: efriedma, craig.topper, spatel, foad, pengfei, lebedev.ri, RKSimonMon, Nov 20, 2:56 AMThis file is larger than 256 KB, so syntax highlighting is disabled by default.