Project

General

Profile

Misc #11795

[PATCH] get rid of breaking strict alias for XL compiler

Added by Zarko (Zarko Todorovski) over 3 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
[ruby-core:71997]

Description

Related to bug 11790. Thank you for adding the changeset.

It may be better to add a IBM XL specific instruction for this:
in line 70 of regparse.h

#ifdef __ibmxl__
 #define SET_NTYPE(node, ntype)  {int value = ntype; memcpy(&((node)->u.base.type), &value, sizeof((node)->u.base.type));}
 #else
 #define SET_NTYPE(node, ntype)   (node)->u.base.type = (ntype)
 #endif

Not sure whether it affects other compilers and it might be safer to only do this with the XL compiler.


Files

ruby_ibmxl.patch (802 Bytes) ruby_ibmxl.patch Zarko (Zarko Todorovski), 12/09/2015 02:29 PM

Related issues

Related to Ruby master - Bug #11790: [PATCH] ANSI alias rules fix ClosedActions

History

Updated by Zarko (Zarko Todorovski) over 3 years ago

Please cancel this, I see I already submitted the changes in bug 11790. Sorry about that.

Updated by nobu (Nobuyoshi Nakada) over 3 years ago

  • Description updated (diff)
  • Status changed from Open to Closed

gcc/clang optimize away that memcpy, at least.

#3

Updated by naruse (Yui NARUSE) over 3 years ago

  • Related to Bug #11790: [PATCH] ANSI alias rules fix added

Also available in: Atom PDF