Fix the slashes when setting the ZapBBInstrDir (#19858)
authorMichelle McDaniel <adiaaida@gmail.com>
Fri, 7 Sep 2018 20:08:25 +0000 (13:08 -0700)
committerGitHub <noreply@github.com>
Fri, 7 Sep 2018 20:08:25 +0000 (13:08 -0700)
commitd81a3685df482e8962bf696e44b9071f19bfac2f
treefd004c0fc82058ed92a3907fc70c967c9dc68623
parent36449cf94237e7cddb7e43a9fe6f873e7f5ebf77
Fix the slashes when setting the ZapBBInstrDir (#19858)

In OpenMethodProfileDataLogFile, we try to set the directory and path for the .ibc files using windows slashes (\). This causes this code to fail on Linux, which uses forward slashes. This is particularly a problem when setting COMPlus_ZapBBInstrDir, which takes that environment variable and attempts to find the name of the file using wcsrchr(assemblyPath, '\'). This causes a crash on linux when collecting IBC counts. The fix is to ifdef it for linux to use the correct path separator.

This change also includes a fix to change the code for checking if IbcTuning is set to use == instead of -eq which was causing a failure in the build that was ignored.
build.sh
src/vm/ceeload.cpp