[PATCH] D81114: [OpenMPOpt] initial tests for ICV tracking. Only nthreads is used.

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added a comment.

Thanks! One comment for the FIXME.



================
Comment at: llvm/test/Transforms/OpenMP/icv_tracking.ll:61
+  tail call void @omp_set_num_threads(i32 10) #1
+  %5 = tail call i32 @omp_get_max_threads() #1
+  tail call void @use(i32 %5) #1
----------------
I guess `%5` should be replaced with `min(%3, 10)`, right? We might want to use SCEV to represent the values and combine them.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81114