[PATCH] D75893: [OpenMP] Add !range metadata to loads from omp.(ub/lb)

[Kaushik Kulkarni via Phabricator via llvm-commits]

kaushikcfd marked 8 inline comments as done.
kaushikcfd added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/OpenMPOpt.cpp:249
+            }
+          };
+
----------------
jdoerfert wrote:
> You need to verify there is only a single load/store and the value is an alloca without initializer and the type is as expected.
> verify there is only a single load/store
There might be multiple loads/store to `%omp.ub` arising from `EnsureUpperBound`. So, we only pick the store just before and the load just after the `static_for` invocation.

>  value is an alloca without initializer
Is the "value" begin discussed here `V`?


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

https://reviews.llvm.org/D75893