Closed nathanchance closed 5 years ago
that's ok, the failures were intentional to shake out bugs. Let's:
Sounds good to me!
For the glob characters issue, it sound like there's no opposition to just supporting them: https://bugs.llvm.org/show_bug.cgi?id=41268. No one has gotten around to it yet.
ihex support is coming in D60270, but I haven't had any time in the past few weeks to review it.
(edit: moved this to the bugs that were split off)
We are tracking this in https://github.com/ClangBuiltLinux/linux/issues/478 and https://github.com/ClangBuiltLinux/linux/issues/480, I don't think there is any reason to keep this open.
https://travis-ci.com/ClangBuiltLinux/continuous-integration/jobs/199950114 https://travis-ci.com/ClangBuiltLinux/continuous-integration/jobs/199950118 https://travis-ci.com/ClangBuiltLinux/continuous-integration/jobs/199950120
Stuff along the lines of:
It looks like this is probably caused by
llvm-strip
not supporting the globs inSTUBCOPY_RM-y
, which @rupprecht noted here. These were removed upstream in https://github.com/torvalds/linux/commit/02562d0ca1084a688ac5c92e0e92947f62f13093 because of https://github.com/torvalds/linux/commit/f922c4abdf7648523589abee9460c87f51630d2f. https://github.com/torvalds/linux/commit/f922c4abdf7648523589abee9460c87f51630d2f decently backports to 4.9 and 4.14, not so clean for 4.4. Might be wise to conditionally turn onllvm-strip
.https://travis-ci.com/ClangBuiltLinux/continuous-integration/jobs/199950119 https://travis-ci.com/ClangBuiltLinux/continuous-integration/jobs/199950121
Relevant bits of Makefile:
This was "resolved" upstream by https://github.com/torvalds/linux/commit/5620a0d1aacd554ebebcff373e31107bb1ef7769, which we obviously can't backport. We could apply a local patch that removes the
fw-shipped
rule in the Makefile or fixllvm-objcopy
(if it is even broken).