[PATCH] D130466: [LICM] - Add option to allow data races

Shubham Narlawar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Sep 10 05:19:40 PDT 2022


gsocshubham added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LICM.cpp:114
+static cl::opt<bool>
+    ThreadModelSingle("licm-force-thread-model-single", cl::Hidden,
+                      cl::init(false),
----------------
gsocshubham wrote:
> xbolva00 wrote:
> > You should rename this variable.
> Do you have any suggestion? Probably, `ForceThreadModelSingle`?
I have renamed flag to `SingleThread`.

@xbolva00 - Let me know WDYT


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

https://reviews.llvm.org/D130466



More information about the llvm-commits mailing list