[PATCH] D92097: [RISCV] Basic jump table lowering

[Jessica Clarke via Phabricator via llvm-commits]

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

I agree with @lenary, I think this is fine to land (with the whitespace fix :)) and if it turns out the threshold is too low then we'll soon find out and can adjust accordingly (but I imagine not, 5 feels like a sensible default). Or if it's too high!



================
Comment at: llvm/test/CodeGen/RISCV/jumptable.ll:166
+
+
+define void @above_threshold(i32 %in, i32* %out) nounwind {
----------------
Nit: unwanted extra blank line


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92097