[PATCH] D75893: [OpenMP] Add !range metadata to loads from omp.(ub/lb)
Kaushik Kulkarni via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 10 10:19:29 PDT 2020
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
More information about the llvm-commits
mailing list