[PATCH] D123290: [LoongArch] Add support for selecting constant materializations.

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added inline comments.


================
Comment at: llvm/lib/Target/LoongArch/MCTargetDesc/LoongArchMatInt.cpp:23
+  //
+  int64_t Highest12 = Val >> 52 & 0xFFF;
+  int64_t Higher20 = Val >> 32 & 0xFFFFF;
----------------
Add const if applicable


================
Comment at: llvm/lib/Target/LoongArch/MCTargetDesc/LoongArchMatInt.cpp:44
+
+  if (SignExtend32<1>(Hi20 >> 19) != SignExtend32<20>(Higher20))
+    Insts.push_back(Inst(LoongArch::LU32I_D, SignExtend64<20>(Higher20)));
----------------
The logic looks weird. What does it do?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123290