[PATCH] D127520: [RISCV] Move creation of constant pools from isel to lowering.

[Philip Reames via Phabricator via llvm-commits]

reames accepted this revision.
reames added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:2973
+  // All simm32 constants should be handled by isel.
+  if (isInt<32>(Imm))
+    return Op;
----------------
Minor: This check should be redundant except maybe as a minor compile time win assuming the count threshold just below is at least 2.  


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127520