[PATCH] D156439: [TLI][AArch64] Extend ReplaceWithVeclib to replace vector FREM instructions for scalable vectors

Jolanta Jensen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 11 02:55:26 PDT 2023


jolanta.jensen added inline comments.


================
Comment at: llvm/lib/CodeGen/ReplaceWithVeclib.cpp:44
+                       std::optional<ElementCount> NumElements = std::nullopt,
+                       std::optional<bool> Masked = std::nullopt) {
+  Module *M = I.getModule();
----------------
jolanta.jensen wrote:
> This will malfunction if false is sent as an argument. To be corrected.
Now fixed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156439



More information about the llvm-commits mailing list