Skip to content

Commit e9f799d

Browse files
marxinjon-y
andcommitted
Do not use HAVE_DOS_BASED_FILE_SYSTEM for Cygwin.
PR gcov-profile/94570 * ltmain.sh: Do not define HAVE_DOS_BASED_FILE_SYSTEM for CYGWIN. PR gcov-profile/94570 * coverage.c (coverage_init): Use separator properly. PR gcov-profile/94570 * filenames.h (defined): Do not define HAVE_DOS_BASED_FILE_SYSTEM for CYGWIN. Co-Authored-By: Jonathan Yong <[email protected]>
1 parent 61b58e7 commit e9f799d

File tree

6 files changed

+30
-9
lines changed

6 files changed

+30
-9
lines changed

ChangeLog

+7
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,10 @@
1+
2020-04-17 Martin Liska <[email protected]>
2+
Jonathan Yong <[email protected]>
3+
4+
PR gcov-profile/94570
5+
* ltmain.sh: Do not define HAVE_DOS_BASED_FILE_SYSTEM
6+
for CYGWIN.
7+
18
2020-04-14 Martin Jambor <[email protected]>
29

310
* MAINTAINERS (Reviewers): Add myself as callgraph (IPA) reviewer.

gcc/ChangeLog

+6
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,9 @@
1+
2020-04-17 Martin Liska <[email protected]>
2+
Jonathan Yong <[email protected]>
3+
4+
PR gcov-profile/94570
5+
* coverage.c (coverage_init): Use separator properly.
6+
17
2020-04-16 Peter Bergner <[email protected]>
28

39
PR rtl-optimization/93974

gcc/coverage.c

+6-6
Original file line numberDiff line numberDiff line change
@@ -1200,6 +1200,11 @@ coverage_obj_finish (vec<constructor_elt, va_gc> *ctor)
12001200
void
12011201
coverage_init (const char *filename)
12021202
{
1203+
#if HAVE_DOS_BASED_FILE_SYSTEM
1204+
const char *separator = "\\";
1205+
#else
1206+
const char *separator = "/";
1207+
#endif
12031208
int len = strlen (filename);
12041209
int prefix_len = 0;
12051210

@@ -1216,11 +1221,6 @@ coverage_init (const char *filename)
12161221
of filename in order to prevent file path clashing. */
12171222
if (profile_data_prefix)
12181223
{
1219-
#if HAVE_DOS_BASED_FILE_SYSTEM
1220-
const char *separator = "\\";
1221-
#else
1222-
const char *separator = "/";
1223-
#endif
12241224
filename = concat (getpwd (), separator, filename, NULL);
12251225
if (profile_prefix_path)
12261226
{
@@ -1252,7 +1252,7 @@ coverage_init (const char *filename)
12521252
if (profile_data_prefix)
12531253
{
12541254
memcpy (da_file_name, profile_data_prefix, prefix_len);
1255-
da_file_name[prefix_len++] = '/';
1255+
da_file_name[prefix_len++] = *separator;
12561256
}
12571257
memcpy (da_file_name + prefix_len, filename, len);
12581258
strcpy (da_file_name + prefix_len + len, GCOV_DATA_SUFFIX);

include/ChangeLog

+7
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,10 @@
1+
2020-04-17 Martin Liska <[email protected]>
2+
Jonathan Yong <[email protected]>
3+
4+
PR gcov-profile/94570
5+
* filenames.h (defined): Do not define HAVE_DOS_BASED_FILE_SYSTEM
6+
for CYGWIN.
7+
18
2020-04-01 Maciej W. Rozycki <[email protected]>
29

310
PR lto/94249

include/filenames.h

+2-1
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,8 @@ Foundation, Inc., 51 Franklin Street - Fifth Floor, Boston, MA 02110-1301, USA.
3232
extern "C" {
3333
#endif
3434

35-
#if defined(__MSDOS__) || defined(_WIN32) || defined(__OS2__) || defined (__CYGWIN__)
35+
#if defined(__MSDOS__) || (defined(_WIN32) && ! defined(__CYGWIN__)) || \
36+
defined(__OS2__)
3637
# ifndef HAVE_DOS_BASED_FILE_SYSTEM
3738
# define HAVE_DOS_BASED_FILE_SYSTEM 1
3839
# endif

ltmain.sh

+2-2
Original file line numberDiff line numberDiff line change
@@ -3425,8 +3425,8 @@ int setenv (const char *, const char *, int);
34253425
# define PATH_SEPARATOR ':'
34263426
#endif
34273427
3428-
#if defined (_WIN32) || defined (__MSDOS__) || defined (__DJGPP__) || \
3429-
defined (__OS2__)
3428+
#if (defined (_WIN32) && ! defined(__CYGWIN__)) || defined (__MSDOS__) || \
3429+
defined (__DJGPP__) || defined (__OS2__)
34303430
# define HAVE_DOS_BASED_FILE_SYSTEM
34313431
# define FOPEN_WB "wb"
34323432
# ifndef DIR_SEPARATOR_2

0 commit comments

Comments
 (0)