david-arm added inline comments.
================ Comment at: clang/lib/CodeGen/CGLoopInfo.cpp:751-753 + // they effectively want vectorization disabled. We leave the + // scalable flag unspecified in this case to avoid setting the + // vectorize.enable flag later on. ---------------- sdesmalen wrote: > is that not something to fix in the code that conditionally sets > vectorize.enable later on instead of working around it here? I did originally try to do that, but I had trouble with it and found it broke other places too. It ended up being simpler to fix it here, but I can play around with it again. Even if this is still the simplest solution I can come back with a more detailed explanation at least! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89031/new/ https://reviews.llvm.org/D89031 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
