t; @@ -1,5 +1,5 @@
> /* GNU Objective C Runtime accessors functions
> - Copyright (C) 2010 Free Software Foundation, Inc.
> + Copyright (C) 2010-2013 Free Software Foundation, Inc.
>Contributed by Nicola Pero
>
> This file is part of GCC.
> Index: libobjc/acinclude.m4
&
> The FSF has had my papers since Feb '11, but are stalling trying to decide
> whether to accept my employer's slightly non-standard disclaimer or not.
Ah. That seems a long time even for this kind of things.
Anyhow, this particular patch consisted of exactly 4 casts, so it seems to fall
in the
>> I checked the attached patch, test results at
>> http://gcc.gnu.org/ml/gcc-testresults/2011-10/msg01377.html
>>
>> which are the same as with my suggested patch.
>>
>> Ok for the trunk?
>
> I probably don't have authority to approve this, but looks OK to me.
The libobjc bits are Ok for trunk.
> (I don't have svn write access so I'll need someone else to commit it if
> it's approved.)
I can apply it for you. But ... do you have a copyright assignment in place
for contributions to GCC ? The patch looks small and trivial enough that
I think I can apply it without a signed copyright assi
commit to the 4.6 branch ?
Thanks
On 6 Jun 2011, at 20:22, Nicola Pero wrote:
> This patch fixes PR obj-c++/48275. It's a routine parser ingenuity.
>
> OK to commit ?
>
> Thanks
>
> Index: testsuite/ChangeLog
> =
;subclass_list = NULL;
Index: ChangeLog
===
--- ChangeLog (revision 179996)
+++ ChangeLog (working copy)
@@ -1,6 +1,19 @@
2011-10-14 Nicola Pero
Backport from mainline
+ 2011-10-09 Nicola Pero
+
+ PR libobjc/49883
+
- method_list = method_list->method_next;
}
node = node->next;
}
Index: ChangeLog
===
--- ChangeLog (revision 179967)
+++ ChangeLog (working copy)
@@ -1,3 +1,13 @@
+2011-10-14 Nicola
actions to handle the language. */
Index: ChangeLog
===
--- ChangeLog (revision 179947)
+++ ChangeLog (working copy)
@@ -1,3 +1,8 @@
+2011-10-14 Nicola Pero
+
+ * gengtype.c (files_rules): Added rules for objc/objc-map.h and
+ objc/objc-map.c.
+
2011-10-13 Jakub Jelinek
Index: gcc/objc/ChangeLog
===
--- gcc/objc/ChangeLog (revision 179864)
+++ gcc/objc/ChangeLog (working copy)
@@ -1,3 +1,54 @@
+2011-10-14 Nicola Pero
+
+ * objc-map.h: New file.
+ * objc-map.c: New file.
+ * c
> It isn't a standoff, we can choose to just fix the issue and be compatible,
> if we want.
I guess you're right and I'm probably using the wrong word - English is not my
first language. ;-)
But I meant that they could have made the same choice to be compatible (by
fixing the issue
in their co
>> Unfortunately, the report was correct in that clang is producing incorrect
>> code and
>> abusing the higher bits of the class->info field to store some other
>> information.
>
> The clang folks are pretty responsive. I'd always give them a chance to
> `fix' thier code, before putting hack-
_b can be NULL), and reloads
Index: ChangeLog
===
--- ChangeLog (revision 179711)
+++ ChangeLog (working copy)
@@ -1,3 +1,18 @@
+2011-10-09 Nicola Pero
+
+ PR libobjc/49883
+ * init.c (__objc_exec_class): Work around a bug in clang's code
+ generation. Cl
e (self, "weakPointer", YES);
@}
- initWithPointer:(const void*)p
Index: gcc/ChangeLog
===
--- gcc/ChangeLog (revision 179710)
+++ gcc/ChangeLog (working copy)
@@ -1,3 +1,9 @@
+2011-10-08 Nicola Pero
+
+ PR libobjc/50428
+ * doc/objc.texi
trunk.
Thanks
Index: libobjc/ChangeLog
===
--- libobjc/ChangeLog (revision 177506)
+++ libobjc/ChangeLog (working copy)
@@ -1,5 +1,17 @@
2011-08-06 Nicola Pero
+ PR libobjc/50002
+ * class.c (__objc_update_class
Fixes a typo in a comment in my last patch. Applied to trunk.
Thanks
Index: ChangeLog
===
--- ChangeLog (revision 177505)
+++ ChangeLog (working copy)
@@ -1,5 +1,9 @@
2011-08-06 Nicola Pero
+ * class.c
This patch fixes PR libobjc/49882. Applied to trunk.
Thanks
Index: libobjc/ChangeLog
===
--- libobjc/ChangeLog (revision 177503)
+++ libobjc/ChangeLog (working copy)
@@ -1,3 +1,10 @@
+2011-08-06 Nicola Pero
+
+ PR
g (revision 176090)
+++ objc/ChangeLog (working copy)
@@ -1,3 +1,30 @@
+2011-07-10 Nicola Pero
+
+ * objc-encoding.h (obstack.h): Do not include.
+ (util_obstack, util_firstobj): Do not declare.
+ (encode_field_decl): Updated prototype to return a tree a
> Huh, I see no reason for this rename. It'll just make patches across
> releases harder.
Sure. But any change will make "patches across releases harder" ... does
it mean we can't make any changes - not even in phase 1 ? :-(
The reason I'd like to change the name is that "attribs.c" is meaningl
ic, newbie-unfriendly, shortened names for files. ;-)
Bootstrapped with c,c++,objc,obj-c++,lto,java,fortran,ada,go on Linux i686.
OK to commit ?
Thanks
In gcc/:
2011-06-22 Nicola Pero
* attribs.c: Renamed to attributes.c.
* Makefile.in: Changed all occurrences of attribs.c and
string.
!ATTR_NAME must be in the form 'text' (not '__text__'). IDENT could
!be the identifier for 'text' or for '__text__'. */
! static inline bool
! is_attribute_p (const char *attr_name, const_tree ident)
! {
! gcc_checking_assert (attr_name[0] != '_
mitris
>
>
> [1] http://gcc.gnu.org/wiki/OptimisingGCC
>
>
>
>
> On Tue, 21 Jun 2011, Richard Guenther wrote:
>
>> On Tue, Jun 21, 2011 at 12:17 PM, Nicola Pero
>> wrote:
>>>>> This patch speeds up the C/C++/ObjC/ObjC++ compiler a little bit by
&
>> This patch speeds up the C/C++/ObjC/ObjC++ compiler a little bit by
>> optimizing
>> lookup_attribute() and is_attribute_p(). The main change is that these
>> functions
>> are now inline.
>
> I don't think this is a good idea.
Can you explain why ? You never do in your response :-)
I'm gue
hem; this one is big enough.
PS2: While doing benchmarks, I accidentally benchmarked an older trunk and
couldn't but notice that compiling
gold with the C++ compiler regressed, in terms of performance, by 1.5%
from 2011-05-19 to 2011-06-20.
Index: ChangeLog
=====
(working copy)
@@ -1,3 +1,8 @@
+2011-06-09 Nicola Pero
+
+ * doc/objc.texi (Traditional GNU Objective-C runtime API):
+ Updated.
+
2011-06-08 Alexandre Oliva
* tree-flow-inline.h (op_iter_init): Reject GIMPLE_PHI stmts.
Index: doc/objc.texi
)
+++ ChangeLog (working copy)
@@ -1,3 +1,21 @@
+2011-06-08 Nicola Pero
+
+ * objc-obj-c++-shared/objc-test-suite-next-encode-assist.h
+ (PMETH, arglist_t): Removed.
+ (method_get_number_of_arguments): Removed.
+ (method_get_nth_argument): Removed
> No. The next number is for the next release, and ideally, we want to sit on
> it for another 10 years or so. We offer no compatibility before the RM says,
> done with the release. Up until then, you can twiddle, after that, no more
> changes to the abi, though, you can bug fix it.
Sounds
I also bumped the API version #define marker.
Committed to trunk.
Thanks
Index: ChangeLog
===
--- ChangeLog (revision 174798)
+++ ChangeLog (working copy)
@@ -1,5 +1,9 @@
2011-06-08 Nicola Pero
+ * objc/objc.h
This works around the fact that libtool configuration may change LD
Index: ChangeLog
===
--- ChangeLog (revision 174797)
+++ ChangeLog (working copy)
@@ -1,5 +1,10 @@
2011-06-08 Nicola Pero
+ * configure.ac (VERSION): Bumped to 4:0:0.
+
>> This patch completes the removal of the public part of the
>> Traditional Objective-C runtime API from libobjc.
>>
>> From now on, the only supported API is the "Modern" API. :-)
>
> If you are removing exported functions from libobjc.so, you should
> bump VERSION= in libobjc/configure.ac.
174789)
+++ ChangeLog (working copy)
@@ -1,3 +1,8 @@
+2011-06-08 Nicola Pero
+
+ * objc/README: Updated.
+ * objc-private/selector.h: Updated comments.
+
2011-06-07 Nicola Pero
* sendmsg.c (class_get_instance_method): Removed.
Index: objc/README
unction is called by: objc_msg_lookup, get_imp and
Index: ChangeLog
===
--- ChangeLog (revision 174773)
+++ ChangeLog (working copy)
@@ -1,5 +1,11 @@
2011-06-07 Nicola Pero
+ * sendmsg.c (class_get_instance_method
if ((object != nil) && CLS_ISCLASS (object->class_pointer))
@@ -122,4 +116,3 @@ object_setClass (id object, Class class_)
return old_class;
}
}
-
Index: ChangeLog
===
--- ChangeLog (revision 174768)
+++ ChangeLog (w
Log
===
--- ChangeLog (revision 174767)
+++ ChangeLog (working copy)
@@ -1,5 +1,10 @@
2011-06-07 Nicola Pero
+ * encoding.c (method_get_number_of_arguments): Removed.
+ (method_get_sizeof_arguments): Removed.
+
+2011-06-07 Nicola Pero
+
* class.c (objc_next_class): Remo
, we update the dispatch tables... */
- __objc_update_dispatch_table_for_class (CLASSOF (impostor));
- __objc_update_dispatch_table_for_class (impostor);
-
- return impostor;
-}
Index: ChangeLog
===
--- ChangeLog (revision 174
@@
+2011-06-07 Nicola Pero
+ Iain Sandoe
+
+ * objc-obj-c++-shared/runtime.h (protocol_getMethodDescription):
+ Added code to deal with the case when [Protocol
+ -descriptionForInstanceMethod:] or [Protocol
+ -descriptionForClassMethod:] returns NULL.
+
+2011
separately.
Thanks Iain for the help! :-)
OK to commit ?
Thanks
Index: ChangeLog
===
--- ChangeLog (revision 174760)
+++ ChangeLog (working copy)
@@ -1,3 +1,10 @@
+2011-06-07 Nicola Pero
+ Iain Sandoe
+
+ * objc-obj
> I checked on i686-darwin9 with both m32 and m32/abi=1 the only difference I
> see is the XPASSes
> for m64 torture/forward-1.m for gnu runtime.
That is very comforting; I have committed the (revised) patch.
Thanks a lot for your help testing and reviewing. :-)
> I will try to test on darwin8 d
06-06 Nicola Pero
+
+ PR objc-++/48275
+ * obj-c++.dg/property/cxx-property-1.mm: New.
+ * obj-c++.dg/property/cxx-property-2.mm: New.
+
2011-06-05 Nicola Pero
PR testsuite/49287
Index: testsuite/obj-c++.dg/property/cxx-property
to change things,
information is readily available.
Index: gcc/objc/ChangeLog
===
--- gcc/objc/ChangeLog (revision 174624)
+++ gcc/objc/ChangeLog (working copy)
@@ -1,3 +1,8 @@
+2011-06-05 Nicola Pero
+
+ * objc-act.c (rec
bmit a separate patch.
Index: ChangeLog
===
--- ChangeLog (revision 174655)
+++ ChangeLog (working copy)
@@ -1,3 +1,10 @@
+2011-06-04 Nicola Pero
+
+ * objc.dg/gnu-api-2-objc.m: Fixed testcase. Use log2 of the
+ alignment, not the align
geLog
===
--- ChangeLog (revision 174617)
+++ ChangeLog (working copy)
@@ -1,5 +1,14 @@
2011-06-03 Nicola Pero
+ * Makefile.in (OBJC_DEPRECATED_H): Removed
struct_objc_static_instances.h
+
Nicola Pero
+ * Makefile.in (OBJC_DEPRECATED_H): Removed objc_object_alloc.h.
+ * objc/deprecated/objc_object_alloc.h: Removed.
+ * objc/objc-api.h: Do not include deprecated/objc_object_alloc.h.
+ * objects.c (_objc_object_alloc, _objc_object_dispose
(working copy)
@@ -1,5 +1,11 @@
2011-06-03 Nicola Pero
+ * Makefile.in (OBJC_DEPRECATED_H): Removed METHOD_NULL.h.
+ * objc/objc-api.h: Do not include deprecated/METHOD_NULL.h.
+ * objc/deprecated/METHOD_NULL.h: Removed.
+
+2011-06-03 Nicola Pero
+
* Makefile.in
);
-}
-
abort ();
}
Index: ChangeLog
===
--- ChangeLog (revision 174614)
+++ ChangeLog (working copy)
@@ -1,5 +1,24 @@
2011-06-03 Nicola Pero
+ * Makefile.in (OBJC_DEPRECATED_H): Removed objc_valloc.h,
+
\
Protocol.h \
- STR.h \
objc_get_uninstalled_dtable.h \
objc_malloc.h \
objc_msg_sendv.h \
Index: ChangeLog
===
--- ChangeLog (revision 174610)
+++ ChangeLog (working copy)
@@ -1,5 +1,11 @@
2011-06-03 Nicola Pero
(working copy)
@@ -1,5 +1,25 @@
2011-06-03 Nicola Pero
+ * Makefile.in (OBJC_H): Removed hash.h and sarray.h.
+ (OBJC_DEPRECATED_H): Likewise.
+ * libobjc.def (objc_hash_new, objc_hash_delete, objc_hash_add,
+ objc_hash_remove, objc_hash_next, objc_hash_value_for_key
, SEL sel, arglist_t args)
{
Index: ChangeLog
===
--- ChangeLog (revision 174594)
+++ ChangeLog (working copy)
@@ -1,5 +1,11 @@
2011-06-03 Nicola Pero
+ * Object.m ([-forward::]): Removed.
+ * objc/deprecated/Obj
.
Thanks
Index: gcc/testsuite/ChangeLog
===
--- gcc/testsuite/ChangeLog (revision 174593)
+++ gcc/testsuite/ChangeLog (working copy)
@@ -1,5 +1,9 @@
2011-06-03 Nicola Pero
+ * objc.dg/headers.m: Do not include objc-list.h
;
-}
Index: libobjc/ChangeLog
===
--- libobjc/ChangeLog (revision 174585)
+++ libobjc/ChangeLog (working copy)
@@ -1,3 +1,19 @@
+2011-06-02 Nicola Pero
+
+ * Makefile.in (OBJC_DEPRECATED_H): Removed objc_error.h.
+
I find confusing and hard to read (and react to) for various reasons.
--
I added testcases with enumerations of cases and combinations to make sure I
was covering the various complications. I
did add a fair amount of comments to the code as well. :-)
Ok to commit to trunk ?
Thanks
Index: objc/ChangeLog
===
l-target-libiberty
configure-target-libstdc++-v3: maybe-configure-target-libgomp
Index: ChangeLog
===
--- ChangeLog (revision 174141)
+++ ChangeLog (working copy)
@@ -1,3 +1,8 @@
+2011-05-27 Nicola Pero
+
+ * Makefile.in
(revision 174335)
+++ objc/ChangeLog (working copy)
@@ -1,3 +1,8 @@
+2011-05-27 Nicola Pero
+
+ * objc-act.c (objc_decl_method_attributes): Implement nonnull
+ attribute for Objective-C methods.
+
2011-05-21 Nicola Pero
* config-lang.in (g
Applied to trunk.
Thanks
Index: ChangeLog
===
--- ChangeLog (revision 174268)
+++ ChangeLog (working copy)
@@ -1,5 +1,10 @@
2011-05-26 Nicola Pero
+ * sendmsg.c: Reindented part of the file. No non-trivial changes
ert (c);
__objc_send_initialize (c);
Index: ChangeLog
===
--- ChangeLog (revision 174267)
+++ ChangeLog (working copy)
@@ -1,3 +1,8 @@
+2011-05-26 Nicola Pero
+
+ * sendmsg.c (__objc_install_dtable_for_class): Use objc_ge
et_implementation.
+ (__objc_init_install_dtable): Removed.
+ (__objc_install_methods_in_dtable): Updated arguments.
+ (__objc_install_dispatch_table_for_class): Renamed to
+ __objc_install_dtable_for_class and updated to support the new
+ +initialize dispatch table logic.
+ (__objc_update_dispatch_table_for_class): Updated to support the
+ new +initialize dispatch table logic.
+ (__objc_forward): Call get_implementation instead of get_imp.
+ (prepared_dtable_table): New.
+ (__objc_prepare_dtable_for_class): New.
+ (__objc_prepared_dtable_for_class): New.
+ (__objc_get_prepared_imp): New.
+ (__objc_install_prepared_dtable_for_class): New.
+
2011-05-24 Nicola Pero
PR libobjc/48177
>> This patch fixes libobjc/48177. I applied it to trunk.
>>
>> I'd like to apply this patch to the 4.6 branch too. Do I need permission
>> from
>> a Release Manager ?
>
> They are always welcome to chime in, though, in this case the libobjc
> maintainer can approve it.
Thanks Mike
I browse
41)
+++ libobjc/ChangeLog (working copy)
@@ -1,3 +1,10 @@
+2011-05-24 Nicola Pero
+
+ PR libobjc/48177
+ * selector.c (__sel_register_typed_name): Use sel_types_match()
+ instead of strcmp() to compare selector types (Suggestion by
+ Richard Frith-Macdonald ).
+
2011-04-15 R
Ok to commit ?
Thanks
Index: ChangeLog
===
--- ChangeLog (revision 174112)
+++ ChangeLog (working copy)
@@ -1,3 +1,15 @@
+2011-05-24 Nicola Pero
+
+ * diagnostic.c (pedwarn): Updated comment.
+
Index: diagnostic.c
ases
(previously they would crash the compiler).
Ok to commit ?
Thanks
Index: ChangeLog
===
--- ChangeLog (revision 174112)
+++ ChangeLog (working copy)
@@ -1,3 +1,11 @@
+2011-05-24 Nicola Pero
+
+ PR objc/48187
+ * c-parser.c (c_
Maybe we can offer an additional compression format like lzma .xz
which brings down 4.5.3 compressed size from 66MB bz2 to 52MB (with
-7).
But that can be decided separately.
Yes ... on the other hand, this does not address the other issue,
which is the
amount of disk space that you need
-- ChangeLog (revision 173994)
+++ ChangeLog (working copy)
@@ -1,3 +1,9 @@
+2011-05-21 Nicola Pero
+
+ * config-lang.in (gtfiles): Updated order of files to fix building
+ when the gcc/cp directory is missing, as in the case of some
+ release tarballs.
+
2011
> However, did you check that the atomic qualifier is correctly written &
> re-read in the state (I believe you did, otherwise it probably won't
> work). This is needed for plugins using it, or using atomic qualified
> fields of existing (or future) structures.
Yes. String options are written as
t;indent, "");
+ break;
+ }
+
if (!length)
{
if (!UNION_OR_STRUCT_P (t->u.p)
Index: gcc/ChangeLog
===
--- gcc/ChangeLog (revision 173917)
+++ gcc/ChangeLog (working copy)
@@ -1,3 +1,8 @@
+2011-05-20 Nicola Pero
+
+ * gengtype.c (walk_type): Implemented "atomic" GTY option.
+ * doc/gty.texi (GTY Options): Document "atomic" GTY option.
+
2011-05-19 Joseph Myers
* config/arm/arm-fpus.def: New.
> Yes why can't you use vec(int,gc)?
Can you use that in libiberty ? I thought it was a GCC-only thing ?
Thanks
>>> This patch adds a new GTY option, "atomic", which is similar to the
>>> identical option you have with Boehm GC
>>> and which can be used with pointers to inform the GC/PCH machinery that
>>> they point to an area of memory that
>> [...]
>>> This patch basically implements it, but at this st
of this patch, and I want to do some more before I
commit. If anyone has good ideas on how
to perform throughout testing, they are welcome. :-)
Ok to commit ?
Thanks
PS: This patch does not include support for marking root/global variables with
"atomic" (neither manually nor
Can I ask for some clarifications on this patch (applied on 2011-02-28) ?
This patch changed the equality function in a hashtable that holds file names,
making for example
"/" and "\" identical entries on MS-DOS, but it didn't change the hash
function, meaning
"/" and "\" are now identical hasht
Alexandre
thanks for the review :-)
> -pedantic review: how about outputting to a temporary file (say
> cp/cfns.hT) and only renaming to the intended name on success, so that,
> if gperf crashes or we reboot part-way through it, we don't end up with
> a partially-generated file that will seem to
>> Ok?
>
> Ping?
PS: For the maintainer who will (eventually) review this patch,
the latest version, tested and with all the comments and contributions
from Joseph and Mike merged in, is --
http://gcc.gnu.org/ml/gcc-patches/2011-04/msg01930.html
So, that's the one to review.
refinement of the patch with
that refinement added too :-)
Ok to commit ?
Thanks
Index: ChangeLog
===
--- ChangeLog (revision 172860)
+++ ChangeLog (working copy)
@@ -1,3 +1,7 @@
+2011-04-22 Nicola Pero
which is
'true' or
empty, so you can do
ifeq ($(ENABLE_MAINTAINER_RULES), true)
... maintainer rules ...
endif
which is readable, expressive, simple and robust.
Index: ChangeLog
===
--- ChangeLog (revis
(srcdir)/cp/cfns.gperf --output-file $(srcdir)/cp/cfns.h
+.PHONY: rebuild-cp-cfns
+
#^L
# Build hooks:
Index: ChangeLog
=======
--- ChangeLog (revision 172858)
+++ ChangeLog (working copy)
@@ -1,3 +1,13 @@
+2011-04-22 Nicola Pero
+
+ * Make-lang.in (rebuild-cp-cfns): New rule replacing the one
orking copy)
@@ -1,3 +1,9 @@
+2011-04-17 Nicola Pero
+
+ * gengtype-state.c (string_eq): New.
+ (read_state): Use string_eq instead of strcmp when creating the
+ state_ident_tab.
+
2011-04-15 Pat Haugen
* config/rs6000/rs6000.c (call_ABI_of_interest): Call
a big file accessing instance variables lots
of
times) the speedup may be bigger.
Ok to commit ?
Thanks
Index: ChangeLog
===
--- ChangeLog (revision 172511)
+++ ChangeLog (working copy)
@@ -1,5 +1,10 @@
2011-04-15 N
e, tree);
Index: c-family/ChangeLog
===
--- c-family/ChangeLog (revision 172444)
+++ c-family/ChangeLog (working copy)
@@ -1,3 +1,9 @@
+2011-04-15 Nicola Pero
+
+ * c-objc.h (objc_get_interface_ivars): Removed.
+ (objc_detect_field_duplicat
> I recently tried to build with --enable-libobjc-gc. While this worked
> out of the box on Solaris, I ran intro trouble on both Tru64 UNIX and
> Darwin.
>
> [...]
>
> The following patch fixes both issues. [...]
>
> Ok for mainline if both pass?
Yes.
[and by the way, I think you're fixing PR l
/ChangeLog
===
--- gcc/c-family/ChangeLog (revision 172399)
+++ gcc/c-family/ChangeLog (working copy)
@@ -1,3 +1,9 @@
+2011-04-13 Nicola Pero
+
+ * stub-objc.c (objc_declare_protocols): Renamed to
+ objc_declare_protocol.
+ * c-objc.h: Likewise.
+
-13 Nicola Pero
+
+ * stub-objc.c (objc_declare_class): Updated argument name.
+
2011-04-12 Nathan Froyd
* c-common.h (c_common_init_ts): Declare.
Index: c-family/stub-objc.c
===
--- c-family/stub-objc.c
first.
Index: ChangeLog
===
--- ChangeLog (revision 172338)
+++ ChangeLog (working copy)
@@ -1,5 +1,10 @@
2011-04-12 Nicola Pero
+ * objc-act.c (build_keyword_selector): Use get_identifier_with_length
+ inste
location_t, tree);
extern tree objc_build_protocol_expr (tree);
Index: ChangeLog
===
--- ChangeLog (revision 172334)
+++ ChangeLog (working copy)
@@ -1,3 +1,8 @@
+2011-04-12 Nicola Pero
+
+ * c-objc.h (objc_build_message_expr
(revision 172328)
+++ ChangeLog (working copy)
@@ -1,5 +1,12 @@
2011-04-12 Nicola Pero
+ * objc-act.c (comp_proto_with_proto): Do not create and use
+ inefficient temporary argument lists. Compare the arguments
+ directly. (match_proto_with_proto): Removed; incorporated
geLog
===
--- ChangeLog (revision 172255)
+++ ChangeLog (working copy)
@@ -1,3 +1,15 @@
+2011-04-11 Nicola Pero
+
+ Rewritten the hashtable holding interfaces for speed.
+ * objc-act.h (INTERFACE_HASHTABLE_SIZE, interface_hashtable_
===
--- ChangeLog (revision 172254)
+++ ChangeLog (working copy)
@@ -1,3 +1,10 @@
+2011-04-11 Nicola Pero
+
+ * objc-act.c (printable_ivar_name): New.
+ (add_instance_variable): Call printable_ivar_name() when an error
+ message needs to be printed. Do not prepare the instance
-04-10 Nicola Pero
+
+ * objc-act.c (objc_is_class_name, objc_is_id): For efficiency,
+ avoid calling identifier_global_value() multiple times.
+
2011-04-06 Joseph Myers
* objc-act.c: Include c-target.h instead of target.h.
Index: gcc/objc/objc-act.c
>> + /* cp/parser.h gives gt-cp-parser.h for cp/parser.c ! */
>> + { DIR_PREFIX_REGEX "cp/parser\\.h$",
>> + REG_EXTENDED, NULL_REGEX,
>> + "gt-cp-parser.h", "cp/parser.c", NULL_FRULACT },
>
> But cp/parser.c also gets its own gt-cp-parser.h. Won't they
> conflict? Doesn't this mean tha
D, NULL_REGEX,
"gt-objc-objc-act.h", "objc/objc-act.c", NULL_FRULACT },
Index: gcc/ChangeLog
===
--- gcc/ChangeLog (revision 171155)
+++ gcc/ChangeLog (working copy)
@@ -1,3 +1,7 @@
+2011-03-18 Nicola Pero
+
+ * gengtype.c (files_rules):
++ testsuites, and committed them as obvious.
Please let me know if it still doesn't work!
Thanks
Index: ChangeLog
===
--- ChangeLog (revision 170682)
+++ ChangeLog (working copy)
@@ -1,5 +1,10 @@
2011-03-04 Nicola
With your patch and Dominique's change both the objc and obj-c++
testsuites pass
without regressions at -m32/-m64.
Updating the objc/c++ testcase is obvious if everyone is on the same
page. :-)
Yes. Thanks a lot to Jack and Dominique for the helpful review/
testing. :-)
I committed
With your patch and Dominique's change both the objc and obj-c++
testsuites pass
without regressions at -m32/-m64.
Updating the objc/c++ testcase is obvious if everyone is on the same
page. :-)
Yes. Thanks a lot to Jack and Dominique for the helpful review/
testing. :-)
I committed
89 matches
Mail list logo
