[PATCH] D45552: [NFC][LV][LoopUtil] Move LoopVectorizationLegality to Analysis tree

Hideki Saito via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 27 21:19:00 PDT 2018


hsaito added inline comments.


================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:7319
                     "attribute is used.\n");
-    ORE->emit(createMissedAnalysis(Hints.vectorizeAnalysisPassName(),
-                                   "NoImplicitFloat", L)
+    ORE->emit(createLVMissedAnalysis(Hints.vectorizeAnalysisPassName(),
+                                     "NoImplicitFloat", L)
----------------
dcaballe wrote:
> I'm probably missing something but why this renaming is necessary?
You don't want original generic name (static file scope) in llvm namespace that's visible to all other parts of LLVM.


https://reviews.llvm.org/D45552





More information about the llvm-commits mailing list