[llvm] [objdump] Fix dxcontainer tests on big endian host (PR #163427)
Aiden Grossman via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 14 11:57:09 PDT 2025
boomanaiden154 wrote:
> The pre-merge checks here are a bit tricky to grok. AArch64 succeeded, but the Linux x86 failed and the failure message is buried in the middle of an extremely long build log. I did look at the Linux failure before I hit the merge button, but the log ended with no clear indication of which sub-action failed just an opaque python script reported failure. That led me to (incorrectly) assume this was a tooling error not caused by my change.
https://github.com/llvm/llvm-project/actions/runs/18504981382 has a rich representation of the failures. We're still working on trying to make it more discoverable, probably through a comment mechanism similar to the code format action. We're still working on finishing some other stuff up before we get to doing exactly that though.
> It would be nice if we could split the build tasks so that the logs get broken up into smaller pieces where we can reasonably expect that the failures would be at the end instead of the middle.
Splitting things up makes local reproduction harder because now you cannot just run a single step. Big chunks of the log are theoretically collapsible, but there are enough log lines that Github doesn't always render things properly.
https://github.com/llvm/llvm-project/pull/163427
More information about the llvm-commits
mailing list