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

Issue 20820002: Use x86_64_general_operand in a bunch more SI mode insn patterns (Closed)

Created:
7 years, 5 months ago by Roland McGrath
Modified:
7 years, 5 months ago
Reviewers:
khim, sehr
CC:
native-client-reviews_googlegroups.com
Base URL:
http://git.chromium.org/native_client/nacl-gcc.git@master
Visibility:
Public.

Description

Use x86_64_general_operand in a bunch more SI mode insn patterns The x86_64_general_operand predicate applies some important constraints, such as that a symbol_ref cannot be an immediate operand under -fPIC. With the ILP32 model, these constraints matter for SI mode instructions (rather than just DI mode) too, since address values are SI mode. Previously the x86_64_general_operand predicate was used inconsistently, e.g. on add patterns but not on sub and mul patterns. This fix changes all the relevant patterns to use x86_64_general_operand in place of vanilla general_operand. This is consistent with how the -mx32 support handles this issue in the upstream trunk code. BUG= https://code.google.com/p/nativeclient/issues/detail?id=3549 TEST= toolchain trybots R=khim@chromium.org Committed: https://git.chromium.org/gitweb?p=native_client/nacl-gcc.git;a=commit;h=d913bb7

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+36 lines, -36 lines) Patch
M gcc/config/i386/i386.md View 31 chunks +36 lines, -36 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Roland McGrath
7 years, 5 months ago (2013-07-26 20:51:00 UTC) #1
khim
LGTM
7 years, 5 months ago (2013-07-26 21:15:03 UTC) #2
Roland McGrath
7 years, 5 months ago (2013-07-26 23:13:16 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 manually as rd913bb7 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698