1
1
Граф коммитов

211 Коммитов

Автор SHA1 Сообщение Дата
Matthias Jurenz
5c43ae156c Fixed # 3704
This commit was SVN r28967.
2013-07-31 07:38:24 +00:00
Matthias Jurenz
c4a7dded5f Changes to VT:
- configure: Removed double slashes in path names which make trouble when building RPMs on Fedora (see #3688)

This commit was SVN r28924.
2013-07-23 08:12:18 +00:00
Matthias Jurenz
ebf441ba4b Changes to VT: Fixed infinite recursion bug if the verbosity level (env. VT_VERBOSE) is higher or equal to 2
This commit was SVN r28624.
2013-06-13 07:33:22 +00:00
Matthias Jurenz
ba9bc238ee attempt to fix #3627: Pass all configure options from the OMPI top-level configure to the OTF sub-configure
This commit was SVN r28616.
2013-06-12 13:39:21 +00:00
Matthias Jurenz
ef0a080028 Changes to VT:
- fixed compiler warnings when compiling for 32-bit
	- MPI wrapper generator scripts:
		- removed non-posix call to length(array)
		- exit scripts if any statement returns a non-true return value (set -e)

This commit was SVN r28524.
2013-05-15 10:44:51 +00:00
Matthias Jurenz
9a0432632a Changes to VT:
- general:
		- incremented version number to 5.14.4
		- fixed Coverity CIDs: 72002, 72099, 72273, 710580, 710664, 710665, 710666
	- VT libs:
		- fixed "incompatible declaration" errors when building against an MPI-3 implementation
		  Since MPI-3 the C keyword "const" is added to all relevant MPI API parameters
		  (e.g. MPI_Send(void* sendbuf, ...) -> MPI_Send(const void* sendbuf, ...)).
		  Prepending the macro CONST to these parameters which is defined either to "const" (if MPI-3) or to nothing (if MPI-1/2).
		- fixed potential buffer overflow when reading the filter file
		- CUDA tracing:
			- enabled access to CUPTI counters for CUDA tracing via CUPTI
			- enabled GPU memory usage tracing independent of the CUDA API
			- enabled recording of CUDA synchronization and implicit synchronization in blocking CUDA memory copies for CUDA tracing via CUPTI
			- enabled recording of synchronous peer-to-peer CUDA memory copies for CUDA tracing via CUPTI
			- consider CUDA data transfers as not idle for option 'pure_idle'
			- fixed identification of the CUDA device ID for CUDA tracing via CUPTI
			- fixed region filtering for applications using the CUDA runtime API wrapper
	- compiler wrappers: add path to mpi.h to the PDT parser command and preprocessor flags

This commit was SVN r28494.
2013-05-14 14:28:04 +00:00
Matthias Jurenz
90dc8cc191 Fixed Coverity CID 993726 (Use after free)
This commit was SVN r28287.
2013-04-04 06:57:00 +00:00
Matthias Jurenz
5c56f6f857 Fixed compiler warnings
This commit was SVN r28275.
2013-04-03 11:38:19 +00:00
Matthias Jurenz
b95ae12f43 Changes to VT/configure:
- Introduced configure option '--enable-fmpi'. If configuring Open MPI w/o Fortran bindings (--disable-mpi-f77/--disable-mpi-fortran), then this option (i.e. --disable-fmpi) will be implicitly passed to the VT configure to disable MPI Fortran support.

This commit was SVN r28260.
2013-03-28 15:53:03 +00:00
Matthias Jurenz
4134d3be74 Changes to OTF:
- fixed Coverity CIDs: 72269, 743285, 743294, 743295, 993125

This commit was SVN r28231.
2013-03-27 13:37:22 +00:00
Matthias Jurenz
00fd159d02 Changes to VT:
- configure:
		- fixed passing FCFLAGS to the Fortran compiler when building libvt-fmpi
		- temporary disabled memory allocation tracing for multi-threaded applications (introduced in v5.14.1) due to broken counter values when memory is freed in a different thread than where it was allocated
		- do not build the unification library when building with HP/Platform MPI (needs linking libmpiCC, even if the C++ MPI interface isn't used)
		- removed 'libfmpi' from list of MPI Fortran libraries to be searched (causes that 'libvt-fmpi' will be built when configuring for HP/Platform MPI)
		- do not build the MPI Fortran library 'libvt-fmpi' when building VT inside Open MPI
	- Java tracing:
		- minor performance improvment: call vt_java_check_error (now vt_java_error) only if a JVMTI function returns an error

This commit was SVN r28228.
2013-03-27 10:04:33 +00:00
Matthias Jurenz
4fbc105b83 Changes to OTF:
- fixed even more Coverity warnings

This commit was SVN r28198.
2013-03-21 12:55:41 +00:00
Matthias Jurenz
7d4be1cb81 Changes to OTF:
- general: Fixed several Coverity warnings
	- otfprofile: Do not show helptext if '-V' given
	- otfinfo: Fixed counting of collective operations and markers

This commit was SVN r28195.
2013-03-21 11:25:05 +00:00
Matthias Jurenz
cf926da7e1 Changes to VT:
- configure: VPATH building: Moved adding of -I$top_srcdir to CPPFLAGS to the end of configure to prevent using this flags for configure checks
	- VT libs: Fixed potential segmentation fault (infinite recursion) when writing intermediate statistics (VT_STAT_INTV != 0)
	- vtunify: Fixed header include to disable OpenMP in the library version of vtunify (libvt-mpi-unify)
	- fixed several Coverity warnings

This commit was SVN r28187.
2013-03-20 15:22:21 +00:00
Matthias Jurenz
8094a57dc7 Changes to OTF:
- general:
		- incremented version number to 1.12.3
		- updated copyright information (2012->2013)
		- fixed compiler warnings occurred when NDEBUG is defined (--with-platform=optimized)
	- otfprofile, LaTeX output:
		- swapped x & y axis of the Message Data Rate Matrix
		- adapted labels to Vampir

Changes to VT:
	- general:
		- incremented version number to 5.14.3
		- updated copyright information (2012->2013)
		- fixed compiler warnings occurred when NDEBUG is defined (--with-platform=optimized)
	- configure / build system:
		- do not search for the MPI lib. when an MPI compiler wrapper was found (prevents double linkage of the MPI lib. when building vtunify-mpi and vtfilter-mpi)
		- prepend cross-prefix to the compiler wrappers *only* if '--with-cross-prefix[=PREFIX]' is given
		- removed unnecessary sources from libvt-java
	- VT libs:
		- extended inoffical API for manual region tracing: allow specifying a group name
		- plugin counters: Merge a set of post-mortem counters and write them under a single async key. This speeds up the unification.
		- Java tracing:
			- removed multiple asking for the current thread id when recording method enter/leave events
			- replaced hash function (id % prime) by Jenking's recommended one ( id & (hash_table_size-1) )
	- vtdyn: Do not instrument regions where inserting instrumentation would requires using a trap
	- vtsetup: Fixed availability state of the exec tracing feature

This commit was SVN r28162.
2013-03-08 13:50:26 +00:00
Matthias Jurenz
a73f6d0254 Changes to OTF:
- corrected initial library version

This commit was SVN r28015.
2013-02-01 14:27:37 +00:00
Matthias Jurenz
06d922005d Changes to OTF:
- revised fix for OpenType Font conflicts based upon Jeff's comments in CMR #3497)
	- renamed libotf to libopen-trace-format
	- install libraries to LIBDIR (as before, no need for adjusting LD_LIBRARY_PATH)

This commit was SVN r28010.
2013-02-01 13:22:13 +00:00
Matthias Jurenz
ac79eab1a2 Changes to OTF:
- fixed build error when the Python bindings were enabled
	- fixed conflicts with OpenType Fonts:
		- install header files to INCLUDEDIR/otf1
		  (only when building outside VT/Open MPI)
		- install libraries to LIBDIR/otf1
		- renamed otfdump to otfprint

This commit was SVN r27983.
2013-01-30 13:44:07 +00:00
Matthias Jurenz
fd00910616 Changes to VT:
- library wrapping: Prevent calling dlerror, if the memory allocation wrappers are enabled. dlerror calls realloc which would ends up in an infinite recursion.

This commit was SVN r27869.
2013-01-18 16:29:06 +00:00
Jeff Squyres
675a2f5c48 Updates for Automake 1.13.x. Without these changes, Automake 1.13.x
will error out, due to use of the
previously-deprecated-and-now-removed AM_CONFIG_HEADER macro.

This commit was SVN r27790.
2013-01-11 20:20:02 +00:00
Matthias Jurenz
9e47ef47ac Changes to VT:
- fixed build error reported in http://www.open-mpi.org/community/lists/users/2013/01/21081.php:
		- removed typo
		- added configure test for declaration of the global 'environ' variable in unistd.h

This commit was SVN r27770.
2013-01-08 14:30:13 +00:00
Matthias Jurenz
1de078c888 Changes to OTF:
- configure: fixed warnings from automake 1.12.x

Changes to VT:
	- general:
		- incremented version number to 5.14.2
	- configure:
		- fixed warnings from automake 1.12.x
	- VT libs / MPI wrappers:
		- do initialize VT and enter the dummy main function ("user") if MPI_Initialized is the very first event to be recorded (fixed assertion error)
		- leave the dummy main function on the same thread where it is entered (fixes potential stack underflow)

This commit was SVN r27734.
2013-01-02 13:42:17 +00:00
Matthias Jurenz
295b04266b Changes to VT:
- configure: pass the MPI configure options (e.g. --with-mpi-lib, --with-mpi-inc-dir) to the OTF configure, even if MPI compiler wrappers were found

This commit was SVN r27710.
2012-12-19 14:46:16 +00:00
Matthias Jurenz
3509deecc3 Changes to VT:
- compiler wrappers: removed invocation of pdbcomment; it removes essential information from the PDB file for instrumenting functions

This commit was SVN r27687.
2012-12-18 13:41:21 +00:00
Matthias Jurenz
9f9fded4de Changes to OTF:
- otfprofile: fixed build error when using the IBM XL C++ compiler

Changes to VT:
	- configure:
		- use AC_CHECK_TYPES instead of AC_CHECK_DECLS to check for PAPI's long_long type
		- use AC_C_INLINE to check whether C 'inline' is present
	- VT libs:
		- set CUPTI tracing as default, if CUDA runtime wrapper has not been built
		- added a common interface for all NVIDIA CUPTI interfaces (events, activity, callbacks)
		- added support for concurrent kernel tracing (since CUDA 5.0)
		- removed almost unused VT_DEBUG env. variable - replaced calls to vt_debug_msg(DBG_LEVEL,...) by vt_cntl_msg (DBG_LEVEL+10,...)
		- added some more ifdefs to new CUDA 5 features
		- added several guards for internal malloc() and free() calls in CUDA related source files
		- revised memory allocation tracing:
			- intercept memory (de)alloaction functions by library wrapping (replaces deprecated hook technique from the GNU C library)
			- added support for multi-threaded applications
			- added wrapper functions for memalign, posix_memalign, and valloc
		- revised exec,system,fork tracing
			- retitled to "Child Process Execution Tracing"
			- introduced env. variable VT_EXECTRACE (marked VT_LIBCTRACE as deprecated)
			- added wrapper functions for execvpe, fexecve, waitid, wait3, and wait4
		- changes default function group name for
			- memory (de)allocation functions: "MEM" -> "LIBC-MALLOC"
			- I/O functions: "I/O" -> "LIBC-I/O"
			- child process execution: "LIBC" -> "LIBC-EXEC"
	- plugin counter interface:
		- added check for initialized vt_plugin_cntr_info.info
	- vtdyn:
		- added missing header includes for Dyninst 8
	- vtrun:
		- do not preload the Dyninst Runtime library; it is loaded by Dyninst itself

This commit was SVN r27679.
2012-12-17 14:40:07 +00:00
Matthias Jurenz
9355d431cc Changes to OTF:
- Docu:
		- fixed broken references in otfprofile_clustering.tex/pdf
	- otfprofile:
		- removed unnecessary '-k' option from otfshrink command

This commit was SVN r27646.
2012-11-27 14:02:08 +00:00
Matthias Jurenz
e10786c62f Changes to OTF:
- general:
		- incremented version number to 1.12.2
	- OTF library:
		- implemented workaround for handling bogus zlib sync. points during reading
Changes to VT:
	- general:
		- incremented version number ot 5.14.1
	- configure:
		- to prevent conflicts with the libtool RPM, append "/vampirtrace" to ${datadir}, if it doesn't points to the default (=${datarootdir})
		  (Fixes trac:3382)

This commit was SVN r27630.

The following Trac tickets were found above:
  Ticket 3382 --> https://svn.open-mpi.org/trac/ompi/ticket/3382
2012-11-26 08:54:14 +00:00
Matthias Jurenz
6320c4b022 Upgraded VT version to 5.14
This commit was SVN r27607.
2012-11-14 13:29:18 +00:00
Matthias Jurenz
280930e299 Do forward '--with-cuda[-libdir]' arguments to the VT configure
This commit was SVN r27560.
2012-11-05 14:53:00 +00:00
Matthias Jurenz
e643c09dee Changes to OTF:
- otfaux:
		- fixed build error on Solaris and NetBSD (removed -lm from library dependencies)
Changes to VT:
	- vtunify:
		- disable OpenMP parallelization if PGI compiler version < 9 is used (threadprivate not supported)

This commit was SVN r27316.
2012-09-12 09:03:10 +00:00
Matthias Jurenz
f5ffb4783c Changes to VT:
- configure: do not build OpenMP support if CFLAGS contains a compiler flag for disabling handling OpenMP directives (e.g. -fno-openmp, -nomp, -hnoomp)
	  Fixes trac:3117

This commit was SVN r27282.

The following Trac tickets were found above:
  Ticket 3117 --> https://svn.open-mpi.org/trac/ompi/ticket/3117
2012-09-10 14:29:47 +00:00
Matthias Jurenz
ef0e8f859a Changes to OTF:
- general:
		- incremented version number ro 1.11.3openmpi
	- otfaux:
		- fixed build error when using the Oracle compiler on Solaris. (removed usage of too recent rint() function)
		  Fixes trac:3257

This commit was SVN r27279.

The following Trac tickets were found above:
  Ticket 3257 --> https://svn.open-mpi.org/trac/ompi/ticket/3257
2012-09-10 12:59:07 +00:00
Matthias Jurenz
a1f3d63278 Fixed build error when compiling VT within Open MPI using the PGI compiler:
The Open MPI configure automatically adds the -D_REENTRANT flag to CPPFLAGS. This causes that one of the PGI STL headers includes the omp.h header - unfortunately the fake one located in tools/vtwrapper/ instead of the real one. Thus, several OpenMP symbols were undefined and the compiler aborted.

This commit was SVN r27130.
2012-08-24 08:34:24 +00:00
Matthias Jurenz
79a065290c Changes to OTF:
- general:
		- incremented version number to 1.11.2openmpi
	- lib OTFAUX:
		- speed-up messages matching, if no snapshots should be generated

Changes to VT:
	- general:
		- incremented version number to 5.13.1openmpi
	- compiler wrappers:
		- vtnvcc:
			- add path to cuda.h to the PDT parser command
			- exclude *.cu source files from instrumenting with PDT/TAU; the PDT parser is not (yet) able to handle CUDA statements and kernels
	- vtunify:
		- fixed timestamp boundary check for merging asynchronous plugin counters (i.e. async. timestamp must >= process' start timestamp)
		- fixed timestamp conversion from local to global
		- print percentage of message matching bumps (unmatched or reversed messages)
		- inlined key-value list "record handler"
		- minor optimizations in hook for merging async. events:
			- search async. source manager only once per stream
			- don't call writeRecHook_Event() if no async. source key is defined

This commit was SVN r26925.
2012-07-31 08:45:37 +00:00
Matthias Jurenz
e5614013cf Changes to VT:
- configure:
		- added option --with[out]-liberty to enable/disable symbol demangling with libiberty; default: disabled, because many systems don't provide a PIC version of libiberty
		- fixed compiler flags for building Fortran MPI wrapper library

This commit was SVN r26839.
2012-07-23 13:21:19 +00:00
Matthias Jurenz
45dfa0d0b1 Changes to VT:
- added CUDA stream reuse for both, CUDA tracing with CUPTI and CUDART wrapper
	- removed CUDA stream number from thread name, when CUDA stream reuse is enabled
	- disable tracking of MPI communicators, requests, windows, etc. if MPI is initialized with MPI_THREAD_SERIALIZED or MPI_THREAD_MULTIPLE (only MPI function enter/leave events will be recorded)
	- configure:
		- fixed detection of compiler instrumentation type on Cray platforms using the cc compiler wrapper
	- compiler wrappers:
		- fixed preprocessing source files to be parsed by OPARI (add path to empty omp.h to the preprocessor flags to avoid multiple declarations of OpenMP functions, types, etc.)
		- vtnvcc: Remove 'compinst' instrumentation type, if VT is configured with a non-GNU compiler instrumentation support (Fixed "unrecognized option" error)
	- vtdyn:
		- added support for instrumenting outer- and inner loops and its iterations (outer=loops within a function, inner=loops within outer loops)
		- try to get the full prototype of functions to be instrumented
		- consider default filter rules also if no filter file is given
		- fixed potential segfault if adding a filter rule w/o stack bounds
		- print verbose messages on stdout if vtdyn is started from the Dyninst attach library (libvt-dynatt)
	- vtunify:
		- print verbose messages on stderr if vtunify is started automatically from the VT library

This commit was SVN r26836.
2012-07-23 10:15:22 +00:00
Matthias Jurenz
a1c6ab7c08 Changes to OTF:
- fixed RBuffer_searchTime() for compressed files

Changes to VT:
	- rebuild HTML+PDF version of VT user manual
	- fixed compiler instr. entry in configure summary (was empty if no compiler instr. found)

This commit was SVN r26750.
2012-07-05 12:56:07 +00:00
Matthias Jurenz
42d54e7bf7 Pass option --with-openmpi-inside=1.7 to VT's configure to select the correct Fortran MPI lib. (libmpi_mpifh instead of libmpi_f77)
This commit was SVN r26744.
2012-07-04 12:53:38 +00:00
Matthias Jurenz
a579b74ebc Upgraded VT version to 5.13
This commit was SVN r26743.
2012-07-04 12:46:53 +00:00
Brian Barrett
9af72072a3 Use MKDIR_P instead of mkdir_p in Makefiles, as MKDIR_P is the only one
defined in recent versions of AC/AM.

This commit was SVN r26625.
2012-06-21 16:52:37 +00:00
Matthias Jurenz
ee735e5f9c Changes to VT:
- fixed build issue on Solaris 10 (reported in http://www.open-mpi.org/community/lists/users/2012/05/19396.php)

This commit was SVN r26553.
2012-06-05 07:54:58 +00:00
Matthias Jurenz
54d67f3957 Changes to VT:
- corrected Fortran MPI lib. name for OMPI 1.7 (-lmpi_f77 -> -lmpi_mpifh)

This commit was SVN r26477.
2012-05-23 09:06:22 +00:00
Matthias Jurenz
bf7e00b787 - corrected underlying compiler command for mpifort-vt (@FC@ -> vtf90)
- removed useless mpi<f77|f90>-vt-wrapper-data.txt

This commit was SVN r26474.
2012-05-23 08:55:00 +00:00
Jeff Squyres
80fa7cba29 Update VT for the new Fortran wrapper compiler nomenclature
This commit was SVN r26354.
2012-04-27 19:51:18 +00:00
Matthias Jurenz
411f9295d0 Changes to VT:
- compiler wrappers:
		- fixed uninstalling TAUINST.SPEC on MacOS (thanks to Jeff for the patch)

This commit was SVN r26280.
2012-04-18 11:59:28 +00:00
Matthias Jurenz
a40b6dcc42 Changes to VT:
- configure:
		- changed default CUPTI library path to $CUPTI-DIR/lib64
	- VT Libs:
		- corrected prototype of MPI_Get_address in Fortran MPI wrappers (the second parameter should be an MPI_Aint* instead of MPI_Fint*)
		- temporary removed MPI_<Comm|Type|Win>_<get|set>_attr and MPI_Attr_<get|put> from the Fortran MPI wrappers due to missing conversion of the attribute value parameter
	- Docu:
		- latex doc \usepackage[T1]{fontenc} so that _ can be searched and copied
		- smaller font in Environment Variables section
		- some improvements in CUDA section
		- removed GPU idle time as official feature for CUPTI tracing method

This commit was SVN r26161.
2012-03-19 08:51:05 +00:00
Matthias Jurenz
61d2818831 Changes to VT:
- general:
		- added missing entry in ChangeLog
	- vtunify[-mpi]:
		- fixed possibly uninitialized global token for the predefined Node and "All" process groups

This commit was SVN r26147.
2012-03-15 10:58:12 +00:00
Matthias Jurenz
03ea0245f0 Changes to OTF:
- general:
		- corrected OTF version number
	- otfprofile:
		- removed leading '=' from CSV lines to make it loadable into spreadsheets (e.g. Open Office)
		- fixed process naming in CSV output of collective operation statistics
Changes to VT:
	- configure:
		- added *_FOR_BUILD variables to CrayXE's default configure options; required for cross-building
	- VT libs:
		- fixed GPU communication, due to new process ID splitting
		- fixed parsing of PAPI native events in VT_METRICS; use strtok_r instead of strtok which is successively called inPAPI_event_name_to_code
		- added VT_METRICS_SEP to definition comments (-> Vampir's trace info)
	- Docu:
		- fixed link to TAU Reference Guide

This commit was SVN r26137.
2012-03-14 12:36:28 +00:00
Matthias Jurenz
164ee92f22 === VT upgrade to version 5.12.2 ===
Changes to OTF:
	- general:
		- updated copyright information (2011->2012)
	- otfmerge-mpi:
		- use the MPI-2 versions of MPI_Address and MPI_Type_struct
	- otfdump:
		- don't abort when reading events fails - the input tracefile might only have statistics

Changes to VT:
	- general:
		- updated version number to 5.12.2openmpi
		- updated copyright information (2011->2012)
	- configure:
		- added configure switches to enable/disable CUPTI and CUDA wrapping
		- fixed detection of C++ runtime libraries for Cray and PGI v11.x compilers
		- fixed detection of Cray compiler's OpenMP flag
		- fixed detection of MPI_IN_PLACE
		- disable support for RTLD_DEFAULT on CrayX? platforms; it's provided by dlfcn.h but not working
		- added '-force_flat_namespace' to linker flags of compiler wrappers on MacOS (causes that OpenMPI's libmpi_f77 calls the VT MPI wrapper functions - not the original ones)
		- default configure options on Cray platforms: use compiler option '--target=$XTPE_COMPILE_TARGET' only if the environment variable is set
	- VT libs:
		- added support for CUDA tracing via CUPTI callbacks and activities (runtime and driver API, kernels, memory copies, GPU idle time and GPU memory usage)
		- added support for cudaMemcpyDefault and synchronous peer-to-peer memory copies in CUDA library wrapper
		- fixed a bug in CUDA runtime wrapper initialization and thread creation
		- fixed a build bug occurred if CUDA and CUPTI found, but support for library tracing is disabled
		- use stack-allocated char-array when composing vtunify command; on some platforms system() results in exit code 127 when using a dynamically allocated char-array
		- fixed bug in async. counter plugin
		- fixed handling of empty MPI groups (MPI_GROUP_EMPTY)
		- fixed handling of MPI groups implicitly generated by MPI_Win_create
		- fixed conversion from MPI_Fint-arrays to MPI_Aint-arrays in Fortran MPI wrappers
		- fixed order of OpenMP threads based on its id (omp_get_thread_num)
		- fixed parsing of filter file to consider non-rank-specific filter rules appearing after a rank selection for disabling
		- fixed handling of 'errno' in LIBC[-I/O] wrappers for statically linked applications (set application's errno to the errno defined in the external LIBC which is used for calling the real functions)
		- suppress warnings about usage of deprecated MPI functions (OMPI_WANT_MPI_INTERFACE_WARNING=0)
	- vtunify[-mpi]:
		- fixed potential memory corruption during enqueuing recv. messages for p2p message matching
	- vtunify-mpi:
		- use the MPI-2 versions of MPI_Address and MPI_Type_struct
		- removed unused MPI wrappers
		- fixed assertion in p2p message matching which occurred when processing local traces with disabled ranks
	- vtdyn:
		- load user-specified shared libraries (-s SHLIB) into the mutatee before starting the instrumentation; adds support for instrumenting shared libraries which are loaded during runtime
	- compiler wrappers:
		- fixed detection of MPI library linked in the path form (e.g. libmpi.a instead of -lmpi)
		- fixed corrupt library order when using vtnvcc for linking MPI/CUDA mixed program
	- OPARI:
		- fixed Fortran parsing for detecting end of block DO loops

This commit was SVN r26114.
2012-03-08 13:01:30 +00:00
Matthias Jurenz
b4523496c8 Changes to VT: Fixed compiler warning about unused variable when compiling on MacOS
This commit was SVN r26076.
2012-03-01 12:17:27 +00:00