[PATCH] D56148: [PowerPC] Fix machine verify pass error for PATCHPOINT pseudo instruction that bad machine code

Nemanja Ivanovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Dec 29 08:30:36 PST 2018


nemanjai added inline comments.


================
Comment at: llvm/lib/Target/PowerPC/PPCISelLowering.cpp:5098
+  // in EmitInstrWithCustomInserter.
+  if (isSVR4ABI && isPPC64 && !isPatchPoint) 
     Ops.push_back(DAG.getRegister(PPC::X2, PtrVT));
----------------
This condition is a never true if the previous one isn't. So this can be changed to just `if (!isPatchPoint)` and contained within the above `if (isSVR4ABI && isPPC64)`.


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

https://reviews.llvm.org/D56148





More information about the llvm-commits mailing list