RFR (XS): JDK-8058534: Remove HeapRegion::_orig_end
Kim Barrett
kim.barrett at oracle.com
Thu Sep 18 23:25:03 UTC 2014
On Sep 18, 2014, at 6:43 PM, Joseph Provino <joseph.provino at oracle.com> wrote:
>
> Kim, thanks. I made that change you suggested.
>
> Webrev: http://cr.openjdk.java.net/~jprovino/8058534/webrev.01/
Sorry, I missed some problems with the assert message added in HeapRegion::initialize()
gc_implementation/g1/heapRegion.cpp, lines 356-9.
There’s a pair of strings to break a source line:
" should match exactly”
"bottom plus …”
missing a space at the end of the first or beginning of second, else output contains “… exactlybottom …”.
String literals immediately followed by identifiers, with no intervening spaces, will run afoul of C++11 user-defined literals. Although gcc and clang provide CLAs to control C++11 features, my understanding is that MS compiler is mostly rolling C++11 features in as implemented in new versions, generally without such controls. As a result, I would expect such usage to fail to compile on some (possibly still in the future) versions. Suggest putting spaces between string literals and macro identifiers.
More information about the hotspot-gc-dev
mailing list