[PATCH] D41953: [LoopUnroll] Unroll and Jam

[Dave Green via Phabricator via llvm-commits]

dmgreen marked 4 inline comments as done.
dmgreen added inline comments.


================
Comment at: lib/Transforms/Scalar/LoopUnrollAndJamPass.cpp:84
+// Prefix of "llvm.loop.unroll." returns true if we have any unroll metadata.
+static bool HasAnyUnrollPragma(const Loop *L, StringRef Prefix) {
+  if (MDNode *LoopID = L->getLoopID()) {
----------------
SjoerdMeijer wrote:
> This is *almost* identical to GetUnrollMetadata() in LoopUnroll.cpp. Is this something that could be shared somehow? Or would that be too inconvenient?
It's similar, but this version is only testing prefixes, not the whole string.


https://reviews.llvm.org/D41953