[PATCH] D88538: [PowerPC] Materialize the immediate that is Uint32 but not Int32.

Amy Kwan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 1 15:34:44 PDT 2020


amyk added inline comments.


================
Comment at: llvm/lib/Target/PowerPC/PPCISelDAGToDAG.cpp:918
 
+  // If the immediate doen't fit int32 but fits uint32, which means the sign bit
+  // is 1. If we use LI/LIS to load the high bits, LIS/LI will sign-extend the
----------------
s/doen't/doesn't 

s/fits/fits in

s/which means/this means


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88538



More information about the llvm-commits mailing list