RESOLVED FIXED 169597
Wrong condition in offlineasm/risc.rb 
https://bugs.webkit.org/show_bug.cgi?id=169597
Summary Wrong condition in offlineasm/risc.rb
Tomas Popela
Reported 2017-03-14 03:53:36 PDT
Found by Coverity scan: webkitgtk-2.14.5/Source/JavaScriptCore/offlineasm/risc.rb:378: unused_expr: Discarding the result of operator "==" in "node.operands().size() == 2". What was this code intended to accomplish? webkitgtk-2.14.5/Source/JavaScriptCore/offlineasm/risc.rb:378: remediation: Did you intend to make an assertion or assignment? # 376| (not validImmediates.include? node.operands[0].value) and # 377| validImmediates.include? -node.operands[0].value # 378|-> node.operands.size == 2 # 379| if node.opcode =~ /add/ # 380| newOpcode = "sub" + $~.post_match
Attachments
Patch (1.57 KB, patch)
2017-03-14 03:55 PDT, Tomas Popela 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Tomas Popela
Comment 1 2017-03-14 03:55:24 PDT
Created attachment 304371 [details] Patch
Mark Lam
Comment 2 2017-03-14 10:39:53 PDT
Comment on attachment 304371 [details] Patch r=me
Tomas Popela
Comment 3 2017-03-14 22:52:16 PDT
Comment on attachment 304371 [details] Patch Clearing flags on attachment: 304371 Committed r213973: <http://trac.webkit.org/changeset/213973>
Tomas Popela
Comment 4 2017-03-14 22:52:25 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.