[PATCH] D35292: [SLPVectorizer] Add propagateIRFlagsWithOp() function to propagate IRFlags for specific Operation

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 14 01:43:12 PDT 2017


RKSimon added inline comments.


================
Comment at: lib/Transforms/Utils/LoopUtils.cpp:1383
+  auto *Intersection = (OpValue == nullptr) ? cast<Instruction>(VL[0]) :
+                                      cast<Instruction>(OpValue);
+  const unsigned Opcode = Intersection->getOpcode();
----------------
These must be dyn_cast and we must check for null. 


================
Comment at: lib/Transforms/Utils/LoopUtils.cpp:1396
   }
 }
----------------
clang-format this whole function.


https://reviews.llvm.org/D35292





More information about the llvm-commits mailing list