Here's an updated webrev:
http://cr.openjdk.java.net/~ihse/JDK-8189229-fix-superfluous-FillCacheFind-paths/webrev.02
/Magnus
On 2017-10-12 17:00, Magnus Ihse Bursie wrote:
Erik commented off-line that this code was intended to include closed
source code. It was indeed correct prior to the consolidation, but has
not been working since. The code is a performance optimization, and
not about correctness, that's why it has not been detected.
I'll come back with an updated review that will restore the behavior
with respect to closed sources.
/Magnus
On 2017-10-12 16:38, Magnus Ihse Bursie wrote:
When creating the consolidated forest, old paths were rewritten
automatically using pattern matching. Unfortunately, this resulted in
incorrect paths in some places.
Specifically, we do not need to look for source code in
$(TOPDIR)/src/*/$(MODULE).
(Actually, some code history digging later it turns out that, while
this was modified during the consolidation, the code was incorrect
even before.)
Bug: https://bugs.openjdk.java.net/browse/JDK-8189229
WebRev:
http://cr.openjdk.java.net/~ihse/JDK-8189229-fix-superfluous-FillCacheFind-paths/webrev.01
/Magnus
