Ignore:
Timestamp:
Dec 2, 2019, 2:45:25 PM (5 years ago)
Author:
Tadeu Zagallo
Message:

[JSC] Remove BytecodeCacheVersion.h
https://bugs.webkit.org/show_bug.cgi?id=204760

Reviewed by Mark Lam.

Having that as a phony make target causes a lot of unnecessary rebuilds. That was a workaround
the fact that we only need a new cache version when we rebuild CachedTypes.cpp, but there was
no straightforward way to get the current timestamp as an integer at that point. Instead, we now
just use a constexpr function that hashes TIMESTAMP.

  • CMakeLists.txt:
  • DerivedSources-output.xcfilelist:
  • DerivedSources.make:
  • runtime/CachedTypes.cpp:

(JSC::jscBytecodeCacheVersion):
(JSC::GenericCacheEntry::isUpToDate const):

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/JavaScriptCore/DerivedSources.make

    r251959 r253010  
    364364    $(OBJECT_LUT_HEADERS) \
    365365#
    366 
    367 .PHONY : BytecodeCacheVersion.h
    368 
    369 BytecodeCacheVersion.h:
    370         echo "#define JSC_BYTECODE_CACHE_VERSION $(shell date '+%s')" > BytecodeCacheVersion.h
    371 
    372 all : BytecodeCacheVersion.h
Note: See TracChangeset for help on using the changeset viewer.