[PATCH] D150856: [lit] Add %{for-each-file} substitution

Joel E. Denny via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 16 09:35:36 PDT 2023


jdenny added a comment.

In D150856#4428651 <https://reviews.llvm.org/D150856#4428651>, @aaron.ballman wrote:

> In D150856#4428635 <https://reviews.llvm.org/D150856#4428635>, @jdenny wrote:
>
>> In D150856#4428523 <https://reviews.llvm.org/D150856#4428523>, @Endill wrote:
>>
>>> We're also can't chain compiler invocations via `&&`: `RUN: %{run98} && %{run11} && %{run14} && %{run17} && %{run20} && %{run23}`, because having dedicated stdout log per each language mode makes DR test debugging workflow much better.
>>
>> Can you explain that a bit more or point to a reference?  I find lit -vv makes it pretty easy to understand what commands are executed and what output goes with each.
>
> Using `&&` means that an earlier process failure (nonzero return) causes later commands to not be run at all, right?

Yes.  Lit does that anyway when commands appear on separate RUN lines.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150856



More information about the llvm-commits mailing list