averello/swift-mirror
1
0
Fork 0
mirror of https://github.com/apple/swift.git synced 2025-12-14 20:36:38 +01:00
Code Issues Packages Projects Releases Wiki Activity

[Runtime] Move ConcurrentReadableArray's allocate/deallocate functions into Storage. Mark ConcurrentReadableArray as un-copyable, un-assignable, and un-movable. Use SWIFT_MEMORY_ORDER_CONSUME instead of std::memory_order_consume. Make read() pass a const pointer.

Browse Source 
rdar://problem/37173156
This commit is contained in:
Mike Ash
2018-05-15 15:27:33 -04:00
parent 68adaebf52
commit da4fa67d7e
4 changed files with 64 additions and 37 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
stdlib/public/runtime/ProtocolConformance.cpp
View File

@@ -326,7 +326,7 @@ void ConformanceState::verify() const {
// Iterate over all of the sections and verify all of the protocol
// descriptors.
auto &Self = const_cast<ConformanceState &>(*this);
Self.SectionsToScan.read([](ConformanceSection *ptr, size_t count) -> char {
Self.SectionsToScan.read([](const ConformanceSection *ptr, size_t count) -> char {
for (size_t i = 0; i < count; i++) {
for (const auto &Record : ptr[i]) {
Record.get()->verify();
@@ -548,7 +548,7 @@ swift_conformsToProtocolImpl(const Metadata * const type,
// Prepare to scan conformance records.
size_t scannedCount;
auto returnNull = C.SectionsToScan
.read([&](ConformanceSection *ptr, size_t count) -> bool {
.read([&](const ConformanceSection *ptr, size_t count) -> bool {
scannedCount = count;
// Scan only sections that were not scanned yet.
// If we found an out-of-date negative cache entry,
@@ -658,7 +658,7 @@ swift::_searchConformancesByMangledTypeName(Demangle::NodePointer node) {
auto &C = Conformances.get();
return C.SectionsToScan
.read([&](ConformanceSection *ptr, size_t count) -> const TypeContextDescriptor * {
.read([&](const ConformanceSection *ptr, size_t count) -> const TypeContextDescriptor * {
for (size_t i = 0; i < count; i++) {
auto &section = ptr[i];
for (const auto &record : section) {