[PATCH] D69542: Full Restrict Support - single patch

Jeroen Dobbelaere via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Apr 29 03:35:30 PDT 2023


jeroen.dobbelaere added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LICM.cpp:2097
+
         // Stores *of* the pointer are not interesting, only stores *to* the
         // pointer.
----------------
ram-NK wrote:
> Rodinia/backprop benchmark of llvm-test-suite is failed, When I test with this patch. 
> My analysis points to a restrict argument ( %hidden_units in bpnn_train_kernel()), Which is promoted by LICM. 
Thanks for the report. I am currently working on a rebase. Once that is available (and published) I'll be working on this and some other issues that were reported.



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

https://reviews.llvm.org/D69542



More information about the llvm-commits mailing list