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

Issue 3187016: [gcc] Fix optimized expand of ptr diff for 64-bit PIC (Closed)

Created:
10 years, 4 months ago by eaeltsin
Modified:
9 years, 7 months ago
CC:
native-client-reviews_googlegroups.com
Visibility:
Public.

Description

[gcc] Fix optimized expand of ptr diff for 64-bit PIC SImode subtraction instructions use "m" contstraint for one operand and "i" constraint for another. This does not work in 64-bit PIC in nacl, as "i" allows symbol references which get rip-relative addressing. This replaces "i" constraint with "e" contraint, where the latter does not allow symbol references except those that simply fit in 32-bits immediate.

Patch Set 1 #

Patch Set 2 : improved comment #

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

Messages

Total messages: 7 (0 generated)
eaeltsin
10 years, 4 months ago (2010-08-23 11:53:49 UTC) #1
khimg
I like the description - it's as cryptic as it gets. Even after you take ...
10 years, 4 months ago (2010-08-23 11:59:31 UTC) #2
eaeltsin
10 years, 4 months ago (2010-08-23 12:11:27 UTC) #3
eaeltsin
OK, Victor, you are right, description sucks. Fixed it together with patch comment. Unfortunately we ...
10 years, 4 months ago (2010-08-23 12:15:25 UTC) #4
pasko-google - do not use
I suspect this may degrade performance of the non-nacl GCC. Can we make specific machine ...
10 years, 4 months ago (2010-08-23 12:22:49 UTC) #5
khimg
2Egor: good idea. We can mark "normal" commands with !TARGET_NACL and add different TARGET_NACL ones ...
10 years, 4 months ago (2010-08-23 12:29:23 UTC) #6
Mark Seaborn
10 years, 4 months ago (2010-08-23 13:06:20 UTC) #7
In the commit message please:
 * Link to the NaCl issue.
 * Say how it was tested.  You can put the small C test case in the commit
message.

Mark

Powered by Google App Engine
This is Rietveld 408576698