[PATCH] D25580: [PowerPC] Improve handling of BUILD_VECTOR nodes (integer results)

Hal Finkel via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 21 08:55:12 PDT 2016


hfinkel added inline comments.


================
Comment at: lib/Target/PowerPC/MCTargetDesc/PPCMCCodeEmitter.cpp:225
   const MCOperand &MO = MI.getOperand(OpNo);
-  if (MO.isImm())
+  // PPC::ZERO and PPC::ZERO8 are register MachineOperand's but they really
+  // represent immediates.
----------------
nemanjai wrote:
> echristo wrote:
> > This seems... weird.
> I agree. But the weirdness exists in the ISA as well. So I'm not sure how best to tackle this issue. Perhaps @hfinkel can provide a pointer regarding this.
This seems weird, in part, because it is not clear why it is necessary to handle this case in a special way. Is there a regression test that covers this? If we have a load with an absolute immediate address, but also a fixup, it is not clear why we'd ignore the fixup.



Repository:
  rL LLVM

https://reviews.llvm.org/D25580





More information about the llvm-commits mailing list