Changeset 60595 in webkit for trunk/JavaScriptCore/pcre

Timestamp:

Jun 2, 2010, 8:15:51 PM (15 years ago)

Author:

[email protected]

Message:

<rdar://problem/8054988> Work around an LLVM GCC code generation bug that results in crashes inside PCRE.

Reviewed by Gavin Barraclough.

• pcre/pcre_exec.cpp:

(repeatInformationFromInstructionOffset): Change the type of instructionOffset to int. There's no good
reason for it to be a short, and using int prevents this code from triggering the LLVM GCC bug.

File:

• trunk/JavaScriptCore/pcre/pcre_exec.cpp (modified) (1 diff)

trunk/JavaScriptCore/pcre/pcre_exec.cpp

@@ -428 +428 @@
 
 // FIXME: "minimize" means "not greedy", we should invert the callers to ask for "greedy" to be less confusing
-static inline void repeatInformationFromInstructionOffset(short instructionOffset, bool& minimize, int& minimumRepeats, int& maximumRepeats)
+static inline void repeatInformationFromInstructionOffset(int instructionOffset, bool& minimize, int& minimumRepeats, int& maximumRepeats)
 {
     // Instruction offsets are based off of OP_CRSTAR, OP_STAR, OP_TYPESTAR, OP_NOTSTAR