WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED DUPLICATE of
bug 228267
229491
postprocess-asm/resolve-asm-file-conflicts.rb build failure with GCC 11.1.0
https://bugs.webkit.org/show_bug.cgi?id=229491
Summary
postprocess-asm/resolve-asm-file-conflicts.rb build failure with GCC 11.1.0
Adrian Perez
Reported
2021-08-25 02:16:40 PDT
Today trying to build WebKitGTK 2.33.3 with GCC 11.1.0 this error was produced (reformatted for clarity): Source/JavaScriptCore/CMakeFiles/LowLevelInterpreterLib.dir/llint/LowLevelInterpreter.cpp.s: Assembler messages: Source/JavaScriptCore/CMakeFiles/LowLevelInterpreterLib.dir/llint/LowLevelInterpreter.cpp.s:114513: Error: file table slot 1 is already occupied by a different file (/home/aperez/webkitgtk-2.33.3/Source/JavaScriptCore/llint/LowLevelInterpreter.cpp vs /home/aperez/webkitgtk-2.33.3/build-gtk//home/aperez/webkitgtk-2.33.3/Source/JavaScriptCore/llint/LowLevelInterpreter.cpp) While this was building 2.33.3, currently the “resolve-asm-conflicts.rb” script is the same in trunk, so we have the issue there as well. I got past the issue with the following, but I think it's not a correct solution at all: --- /home/aperez/WebKit/Source/JavaScriptCore/Scripts/resolve-asm-file-conflicts.rb 2021-03-25 16:41:56.741540337 +0200 +++ ./Source/JavaScriptCore/Scripts/resolve-asm-file-conflicts.rb 2021-08-25 12:07:38.225428021 +0300 @@ -86,8 +87,10 @@ if not @s.eos? return ParseResultError.new("Expected end of line after #{ret2.str}") end - return ParseResultSuccess.new((Pathname.new(ret1.str) / ret2.str).cleanpath.to_s) + # return ParseResultSuccess.new((Pathname.new(ret1.str) / ret2.str).cleanpath.to_s) + return ParseResultSuccess.new(Pathname.new(ret1.str).cleanpath.to_s) end + def parse_string_literal if @s.scan(/"/).nil? err = "Expected string literal at `#{@s.string}` (pos #{@s.pos})" @@ -395,7 +399,7 @@ # First, make sure our tests still pass. This only takes a fraction of # our runtime and ensures the tests will get run by anyone trying out # changes to this file. -selftest +# selftest if ARGV.size != 2 $stderr.puts("Usage: #{$0} input output")
Attachments
Patch for landing
(1.41 KB, patch)
2021-08-30 08:17 PDT
,
Michael Catanzaro
mcatanzaro
: commit-queue-
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Adrian Perez
Comment 1
2021-08-25 02:17:43 PDT
The last time the script was touched was in
bug #223136
so CC'ing Angelos in case he has some idea on how to go about this :)
Michael Catanzaro
Comment 2
2021-08-30 08:17:45 PDT
Unfortunately I didn't notice this because I had added -DGCC_OFFLINEASM_SOURCE_MAP=OFF to my build script in order to facilitate building. Now that I've removed it, I cannot build anymore due to this issue. I'm going to turn this option off for now in an unreviewed commit, to get the build working again for now. I'm bamboozled as to why this didn't break any Fedora or GNOME builds.
Michael Catanzaro
Comment 3
2021-08-30 08:17:58 PDT
Created
attachment 436773
[details]
Patch for landing
Michael Catanzaro
Comment 4
2021-08-30 08:19:29 PDT
Apparently it's already fixed in
bug #228267
. We just forgot to land that patch. *** This bug has been marked as a duplicate of
bug 228267
***
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug