* [Patch ObjC/Committed] fix PR47997 (part 2).
@ 2011-10-29 15:09 Iain Sandoe
0 siblings, 0 replies; only message in thread
From: Iain Sandoe @ 2011-10-29 15:09 UTC (permalink / raw)
To: GCC Patches; +Cc: Mike Stump, Nicola Pero
[-- Attachment #1: Type: text/plain, Size: 36 bytes --]
As approved on the PR thread,
Iain
[-- Attachment #2: 180650-pr467997-objc-diff.txt --]
[-- Type: text/plain, Size: 1123 bytes --]
Index: gcc/objc/ChangeLog
===================================================================
--- gcc/objc/ChangeLog (revision 180650)
+++ gcc/objc/ChangeLog (working copy)
@@ -1,3 +1,10 @@
+2011-10-29 Iain Sandoe <iains@gcc.gnu.org>
+
+ PR target/47997
+ * objc-act.c (objc_build_string_object): Remove redundant second
+ call to fix_string_type (). Add a checking assert that we are,
+ indeed, passed a STRING_CST.
+
2011-10-18 Mikael Pettersson <mikpe@it.uu.se>
PR objc/50743
Index: gcc/objc/objc-act.c
===================================================================
--- gcc/objc/objc-act.c (revision 180650)
+++ gcc/objc/objc-act.c (working copy)
@@ -3128,9 +3128,8 @@ objc_build_string_object (tree string)
struct string_descriptor *desc, key;
void **loc;
- /* Prep the string argument. */
- string = fix_string_type (string);
- TREE_SET_CODE (string, STRING_CST);
+ /* We should be passed a STRING_CST. */
+ gcc_checking_assert (TREE_CODE (string) == STRING_CST);
length = TREE_STRING_LENGTH (string) - 1;
/* The target may have different ideas on how to construct an ObjC string
[-- Attachment #3: Type: text/plain, Size: 1 bytes --]
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2011-10-29 13:01 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-10-29 15:09 [Patch ObjC/Committed] fix PR47997 (part 2) Iain Sandoe
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).