Jan-Simon Möller
2014-May-16 13:01 UTC
[LLVMdev] [LLVMLinux] Regression: rev 208833/208834 break linux kernel build in ASM handling
Hi ! Our buildbot found this regression while compiling the kernel with clang: A bisection points to 475ac5d302ba84ac13d34a9215c29c1a38ca5690 is the first bad commit commit 475ac5d302ba84ac13d34a9215c29c1a38ca5690 Author: Eric Christopher <echristo at gmail.com> Date: Thu May 15 01:10:50 2014 +0000 Unify command line handling of MCTargetOptions and remove extra options and code. No functional change. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk at 208833 91177308-0d34-0410-b5e6-96231b3b80d8 8454a4b9373bca5d2a8f596ef1c33826171729bd is the first bad commit commit 8454a4b9373bca5d2a8f596ef1c33826171729bd Author: Eric Christopher <echristo at gmail.com> Date: Thu May 15 01:21:56 2014 +0000 Fill in the options in the MCOptions struct directly like the other options. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk at 208834 91177308-0d34-0410-b5e6-96231b3b80d8 Before: http://buildbot.llvm.linuxfoundation.org/builders/2_llvm/builds/8/steps/shell_3/logs/stdio After: http://buildbot.llvm.linuxfoundation.org/builders/2_llvm/builds/41/steps/shell_3/logs/stdio --- snip --- /tmp/vfpmodule-0d3341.s: Assembler messages: /tmp/vfpmodule-0d3341.s:64: Error: junk at end of line, first unrecognized character is `"' /tmp/vfpmodule-0d3341.s:65: Error: junk at end of line, first unrecognized character is `"' /tmp/vfpmodule-0d3341.s:66: Error: junk at end of line, first unrecognized character is `"' --- snip --- Current state can be tracked: http://buildbot.llvm.linuxfoundation.org/waterfall?show=2_llvm Can we fix or revert pls ? -- Dipl.-Ing. Jan-Simon Möller jansimon.moeller at gmx.de
Renato Golin
2014-May-16 13:25 UTC
[LLVMdev] [LLVMLinux] Regression: rev 208833/208834 break linux kernel build in ASM handling
On 16 May 2014 14:01, Jan-Simon Möller <dl9pf at gmx.de> wrote:> A bisection points to > git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk at 208833 > git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk at 208834Hi Jan, These commits really seem to have no impact whatsoever (default is still false, no changes in default command line options).> --- snip --- > /tmp/vfpmodule-0d3341.s: Assembler messages: > /tmp/vfpmodule-0d3341.s:64: Error: junk at end of line, first unrecognized > character is `"' > /tmp/vfpmodule-0d3341.s:65: Error: junk at end of line, first unrecognized > character is `"' > /tmp/vfpmodule-0d3341.s:66: Error: junk at end of line, first unrecognized > character is `"' > --- snip ---Can you send the contents of the temporary ASM file, so we have an idea what's the offending junk? cheers, --renato
Jan-Simon Möller
2014-May-16 14:32 UTC
[LLVMdev] [LLVMLinux] Regression: rev 208833/208834 break linux kernel build in ASM handling
Hi ! I reproduced it on the file init/main.c The invocation, log and main.i / main.s is attached. -- Dipl.-Ing. Jan-Simon Möller jansimon.moeller at gmx.de Am Freitag, 16. Mai 2014, 14:25:47 schrieb Renato Golin:> On 16 May 2014 14:01, Jan-Simon Möller <dl9pf at gmx.de> wrote: > > A bisection points to > > > > git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk at 208833 > > git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk at 208834 > > Hi Jan, > > These commits really seem to have no impact whatsoever (default is > still false, no changes in default command line options). > > > --- snip --- > > /tmp/vfpmodule-0d3341.s: Assembler messages: > > /tmp/vfpmodule-0d3341.s:64: Error: junk at end of line, first unrecognized > > character is `"' > > /tmp/vfpmodule-0d3341.s:65: Error: junk at end of line, first unrecognized > > character is `"' > > /tmp/vfpmodule-0d3341.s:66: Error: junk at end of line, first unrecognized > > character is `"' > > --- snip --- > > Can you send the contents of the temporary ASM file, so we have an > idea what's the offending junk? > > cheers, > --renato-------------- next part -------------- A non-text attachment was scrubbed... Name: main.c.clang.log Type: text/x-log Size: 18661 bytes Desc: not available URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20140516/427e46e4/attachment.bin> -------------- next part -------------- A non-text attachment was scrubbed... Name: main.i.bz2 Type: application/x-bzip Size: 175060 bytes Desc: not available URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20140516/427e46e4/attachment-0001.bin> -------------- next part -------------- A non-text attachment was scrubbed... Name: main.s.bz2 Type: application/x-bzip Size: 93797 bytes Desc: not available URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20140516/427e46e4/attachment-0002.bin>
Nick Lewycky
2014-May-16 19:40 UTC
[LLVMdev] [LLVMLinux] Regression: rev 208833/208834 break linux kernel build in ASM handling
On 16 May 2014 06:01, Jan-Simon Möller <dl9pf at gmx.de> wrote:> Hi ! > > Our buildbot found this regression while compiling the kernel with clang: > > A bisection points to > > 475ac5d302ba84ac13d34a9215c29c1a38ca5690 is the first bad commit > commit 475ac5d302ba84ac13d34a9215c29c1a38ca5690 > Author: Eric Christopher <echristo at gmail.com> > Date: Thu May 15 01:10:50 2014 +0000 > > Unify command line handling of MCTargetOptions and remove extra > options and code. No functional change. > > git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk at 208833 > 91177308-0d34-0410-b5e6-96231b3b80d8 > > 8454a4b9373bca5d2a8f596ef1c33826171729bd is the first bad commit > commit 8454a4b9373bca5d2a8f596ef1c33826171729bd > Author: Eric Christopher <echristo at gmail.com> > Date: Thu May 15 01:21:56 2014 +0000 > > Fill in the options in the MCOptions struct directly like the other > options. > > git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk at 208834 > 91177308-0d34-0410-b5e6-96231b3b80d8 > > > Before: > > http://buildbot.llvm.linuxfoundation.org/builders/2_llvm/builds/8/steps/shell_3/logs/stdio > > After: > > http://buildbot.llvm.linuxfoundation.org/builders/2_llvm/builds/41/steps/shell_3/logs/stdio > > --- snip --- > /tmp/vfpmodule-0d3341.s: Assembler messages: > /tmp/vfpmodule-0d3341.s:64: Error: junk at end of line, first unrecognized > character is `"' > /tmp/vfpmodule-0d3341.s:65: Error: junk at end of line, first unrecognized > character is `"' > /tmp/vfpmodule-0d3341.s:66: Error: junk at end of line, first unrecognized > character is `"' > --- snip --- >I found the same thing. The problem is that the 3-arg form of .file is now backwards w.r.t. setting of integrated assembler. To reproduce: $ clang -no-integrated-as -S a/x.c -o - -g | grep file .file 1 "a" "x.c" vs. $ clang -S a/x.c -o - -g | grep file .file 1 "a/x.c" Binutils doesn't support the 3-arg format, so unless we know we're using the integrated assembler we can't emit that. As I recall, clang -S would emit the 2-argument form unless you passed -integrated-as -S on the command-line. Nick> Current state can be tracked: > http://buildbot.llvm.linuxfoundation.org/waterfall?show=2_llvm > > > Can we fix or revert pls ? > > -- > > Dipl.-Ing. > Jan-Simon Möller > > jansimon.moeller at gmx.de > > _______________________________________________ > LLVM Developers mailing list > LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu > http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev >-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20140516/3cee943e/attachment.html>
Eric Christopher
2014-May-16 21:13 UTC
[LLVMdev] [LLVMLinux] Regression: rev 208833/208834 break linux kernel build in ASM handling
Should be fixed now. -eric On Fri, May 16, 2014 at 12:40 PM, Nick Lewycky <nlewycky at google.com> wrote:> On 16 May 2014 06:01, Jan-Simon Möller <dl9pf at gmx.de> wrote: > >> Hi ! >> >> Our buildbot found this regression while compiling the kernel with clang: >> >> A bisection points to >> >> 475ac5d302ba84ac13d34a9215c29c1a38ca5690 is the first bad commit >> commit 475ac5d302ba84ac13d34a9215c29c1a38ca5690 >> Author: Eric Christopher <echristo at gmail.com> >> Date: Thu May 15 01:10:50 2014 +0000 >> >> Unify command line handling of MCTargetOptions and remove extra >> options and code. No functional change. >> >> git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk at 208833 >> 91177308-0d34-0410-b5e6-96231b3b80d8 >> >> 8454a4b9373bca5d2a8f596ef1c33826171729bd is the first bad commit >> commit 8454a4b9373bca5d2a8f596ef1c33826171729bd >> Author: Eric Christopher <echristo at gmail.com> >> Date: Thu May 15 01:21:56 2014 +0000 >> >> Fill in the options in the MCOptions struct directly like the other >> options. >> >> git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk at 208834 >> 91177308-0d34-0410-b5e6-96231b3b80d8 >> >> >> Before: >> >> http://buildbot.llvm.linuxfoundation.org/builders/2_llvm/builds/8/steps/shell_3/logs/stdio >> >> After: >> >> http://buildbot.llvm.linuxfoundation.org/builders/2_llvm/builds/41/steps/shell_3/logs/stdio >> >> --- snip --- >> /tmp/vfpmodule-0d3341.s: Assembler messages: >> /tmp/vfpmodule-0d3341.s:64: Error: junk at end of line, first unrecognized >> character is `"' >> /tmp/vfpmodule-0d3341.s:65: Error: junk at end of line, first unrecognized >> character is `"' >> /tmp/vfpmodule-0d3341.s:66: Error: junk at end of line, first unrecognized >> character is `"' >> --- snip --- >> > > I found the same thing. > > The problem is that the 3-arg form of .file is now backwards w.r.t. > setting of integrated assembler. To reproduce: > > $ clang -no-integrated-as -S a/x.c -o - -g | grep file > .file 1 "a" "x.c" > > vs. > > $ clang -S a/x.c -o - -g | grep file > .file 1 "a/x.c" > > Binutils doesn't support the 3-arg format, so unless we know we're using > the integrated assembler we can't emit that. As I recall, clang -S would > emit the 2-argument form unless you passed -integrated-as -S on the > command-line. > > Nick > >> Current state can be tracked: >> http://buildbot.llvm.linuxfoundation.org/waterfall?show=2_llvm >> >> >> Can we fix or revert pls ? >> >> -- >> >> Dipl.-Ing. >> Jan-Simon Möller >> >> jansimon.moeller at gmx.de >> >> _______________________________________________ >> LLVM Developers mailing list >> LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu >> http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev >> > >-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20140516/e7e94e60/attachment.html>