[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
Sun Mar 15 13:23:34 PDT 2020


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


================
Comment at: llvm/lib/Transforms/IPO/OpenMPOpt.cpp:235
+    bool Changed = false;
+    RuntimeFunctionInfo &RFI = RFIs[OMPRTL___kmpc_for_static_init_4];
+
----------------
jdoerfert wrote:
> In this patch or a follow up we need to do this for the other loop init runtime calls as well (there are 4 or 8 I think).
I would prefer I do that in a followup patch. It would be easier if we keep this patch only for checking the correctness of logic behind setting the range of bounds.


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

https://reviews.llvm.org/D75893





More information about the llvm-commits mailing list