[PATCH] D64916: [LV] isScalarEpilogueAllowed. NFC.

Hideki Saito via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 24 10:14:52 PDT 2019


hsaito added a comment.

Just one comment from me.

@Meinersbur, you think your comments are addressed enough?



================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:1290
   /// iterations to execute in the scalar loop.
-  bool IsScalarEpilogueAllowed = true;
+  ScalarEpilogueLowering IsScalarEpilogueAllowed = CM_ScalarEpilogueAllowed;
 
----------------
IsSomething sounds like a boolean value. Suggest something like ScalarEpilogueStatus, ScalarEpilogueSetting, etc.


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

https://reviews.llvm.org/D64916





More information about the llvm-commits mailing list