[libcxx-commits] [PATCH] D144575: [libc++][CI] Improves Dockerfile

[Nikolas Klauser via Phabricator via libcxx-commits]

philnik added inline comments.


================
Comment at: libcxx/utils/ci/Dockerfile:67-71
+    # previous release
+    bash /tmp/llvm.sh $(($LLVM_HEAD_VERSION - 2)) && \
+    # latest release
+    bash /tmp/llvm.sh $(($LLVM_HEAD_VERSION - 1)) && \
+    # current ToT
----------------
Mordante wrote:
> philnik wrote:
> > 
> I like to keep the comments for clarity.
I really don't see how describing what the lines do on-by-one help in any way. They just make it harder to read IMO.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144575