experiments/STT-tensorflow
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Better workaround for MSVC 14.0 limitation related to constexpr array.

Browse Source 
PiperOrigin-RevId: 258793545
This commit is contained in:
Christian Sigg 2019-07-18 10:10:36 -07:00 committed by TensorFlower Gardener
parent f6e0ec468a
commit 6ab65bc0c9
2 changed files with 5 additions and 18 deletions

9
tensorflow/core/util/memmapped_file_system.cc
View File

@ -190,13 +190,8 @@ const void* MemmappedFileSystem::GetMemoryWithOffset(uint64 offset) const {
return reinterpret_cast<const uint8*>(mapped_memory_->data()) + offset;
}
#if defined(_MSC_VER)
constexpr char* MemmappedFileSystem::kMemmappedPackagePrefix;
constexpr char* MemmappedFileSystem::kMemmappedPackageDefaultGraphDef;
#else
constexpr char MemmappedFileSystem::kMemmappedPackagePrefix[];
constexpr char MemmappedFileSystem::kMemmappedPackageDefaultGraphDef[];
#endif
constexpr const char MemmappedFileSystem::kMemmappedPackagePrefix[];
constexpr const char MemmappedFileSystem::kMemmappedPackageDefaultGraphDef[];
Status MemmappedFileSystem::InitializeFromFile(Env* env,
const string& filename) {

14
tensorflow/core/util/memmapped_file_system.h
View File

@ -53,19 +53,11 @@ class MemmappedFileSystem : public FileSystem {
public:
// Memmapped regions use this prefix to distinguish from
// the filesystem.
#if defined(_MSC_VER)
static constexpr char* kMemmappedPackagePrefix =
#else
static constexpr char kMemmappedPackagePrefix[] =
#endif
static constexpr const char kMemmappedPackagePrefix[] =
"memmapped_package://";
// The default graphdef in the package.
#if defined(_MSC_VER)
static constexpr char* kMemmappedPackageDefaultGraphDef =
#else
static constexpr char kMemmappedPackageDefaultGraphDef[] =
#endif
// The default graphdef in the package.
static constexpr const char kMemmappedPackageDefaultGraphDef[] =
"memmapped_package://.";
MemmappedFileSystem();