Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(626)

Side by Side Diff: src/trusted/validator_mips/validation-report.py

Issue 1535443005: [MIPS] Make validator allow only two load patterns via $t8 (Closed) Base URL: https://chromium.googlesource.com/native_client/src/native_client@master
Patch Set: Minor update. Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 #!/usr/bin/python 1 #!/usr/bin/python
2 # 2 #
3 # Copyright (c) 2012 The Native Client Authors. All rights reserved. 3 # Copyright (c) 2012 The Native Client Authors. All rights reserved.
4 # Use of this source code is governed by a BSD-style license that can be 4 # Use of this source code is governed by a BSD-style license that can be
5 # found in the LICENSE file. 5 # found in the LICENSE file.
6 # 6 #
7 7
8 import sys 8 import sys
9 import os 9 import os
10 import textwrap 10 import textwrap
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
44 'kProblemPatternCrossesBundle': ['This instruction is part of a ' 44 'kProblemPatternCrossesBundle': ['This instruction is part of a '
45 'sequence that must execute in full, but it spans a bundle edge ' 45 'sequence that must execute in full, but it spans a bundle edge '
46 '-- so an indirect branch may target it', 46 '-- so an indirect branch may target it',
47 1, 1], 47 1, 1],
48 'kProblemBranchInvalidDest': ['This branch targets a location that is ' 48 'kProblemBranchInvalidDest': ['This branch targets a location that is '
49 'outside of the application\'s executable code, over 256 MB', 49 'outside of the application\'s executable code, over 256 MB',
50 0, 0], 50 0, 0],
51 'kProblemUnsafeLoadStore': ['This load/store instruction is not ' 51 'kProblemUnsafeLoadStore': ['This load/store instruction is not '
52 'preceded by a valid store mask instruction', 52 'preceded by a valid store mask instruction',
53 1, 0], 53 1, 0],
54 'kProblemUnsafeLoadStoreThreadPointer': ['This is not a valid '
55 'load/store instruction for accessing thread pointer.',
56 0, 0],
54 'kProblemUnsafeJumpRegister': ['This indirect jump instruction is not ' 57 'kProblemUnsafeJumpRegister': ['This indirect jump instruction is not '
55 'preceded by a valid jump mask instruction', 58 'preceded by a valid jump mask instruction',
56 1, 0], 59 1, 0],
57 'kProblemUnsafeDataWrite': ['This instruction affects a register that ' 60 'kProblemUnsafeDataWrite': ['This instruction affects a register that '
58 'must contain a valid data-region address (sp), but is not ' 61 'must contain a valid data-region address (sp), but is not '
59 'followed by a valid store mask instruction', 62 'followed by a valid store mask instruction',
60 0, 1], 63 0, 1],
61 'kProblemReadOnlyRegister': ['This instruction changes the contents of' 64 'kProblemReadOnlyRegister': ['This instruction changes the contents of'
62 ' read-only register', 65 ' read-only register',
63 0, 0], 66 0, 0],
(...skipping 56 matching lines...) Expand 10 before | Expand all | Expand 10 after
120 return 1 123 return 1
121 for line in sys.stdin: 124 for line in sys.stdin:
122 if line.startswith('ncval: '): 125 if line.startswith('ncval: '):
123 line = line[7:].strip() 126 line = line[7:].strip()
124 _explain_problem(sys.argv[1], *_parse_report(line)) 127 _explain_problem(sys.argv[1], *_parse_report(line))
125 return 0 128 return 0
126 129
127 130
128 if __name__ == '__main__': 131 if __name__ == '__main__':
129 sys.exit(main(sys.argv)) 132 sys.exit(main(sys.argv))
OLDNEW
« no previous file with comments | « src/trusted/validator_mips/testdata/test_tp_access.err ('k') | src/trusted/validator_mips/validator.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698