[PATCH] D110616: [Legalizer] Expand BR_CC into SETCC if condition code is legal

Qiu Chaofan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 16 20:34:27 PST 2021


qiucf added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp:3562
                          Node->getOperand(2));
+      Results.push_back(Tmp1);
     } else {
----------------
craig.topper wrote:
> Why did the push_backs need to be moved?
Here when `BR_CC` is not legal, `Results` should be empty.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110616



More information about the llvm-commits mailing list