================
@@ -29,8 +29,8 @@ mlir::Value fir::runtime::genInitArrayConstructorVector(
   // the target. The "cookieSize" argument is used to validate this wild
   // assumption until runtime interfaces are improved.
----------------
jeanPerier wrote:

It seems like you can get rid of the cookieSize argument and comments since the 
validations are done via the added static_assert inside the runtime (which 
seems better, thanks).

https://github.com/llvm/llvm-project/pull/112188
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
  • [... Michael Kruse via llvm-branch-commits
    • ... Michael Kruse via llvm-branch-commits
    • ... Michael Kruse via llvm-branch-commits
    • ... via llvm-branch-commits
    • ... Valentin Clement バレンタイン クレメン via llvm-branch-commits
    • ... Michael Kruse via llvm-branch-commits
    • ... Michael Kruse via llvm-branch-commits
    • ... via llvm-branch-commits
    • ... via llvm-branch-commits
    • ... Peter Klausler via llvm-branch-commits
    • ... Michael Kruse via llvm-branch-commits
    • ... Michael Kruse via llvm-branch-commits
    • ... Valentin Clement バレンタイン クレメン via llvm-branch-commits
    • ... Michael Kruse via llvm-branch-commits
    • ... Michael Kruse via llvm-branch-commits
    • ... Michael Kruse via llvm-branch-commits
    • ... Michael Kruse via llvm-branch-commits
    • ... Michael Kruse via llvm-branch-commits

Reply via email to