Update memory accounting

[akleshchev]

1. Properly track commit charge on windows
2. Report memory stats to bugsplat
3. Fix virtual memory overflow

[Copilot]

Pull request overview

This PR updates the viewer’s memory accounting and reporting to better reflect Windows commit charge, add crash-time memory stats to BugSplat reports, and reduce risk of memory-stat overflows by switching certain accounting to MB units.

Changes:

• Add Windows-only commit-availability tracking and expose it to texture memory-pressure logic.
• Cache memory queries (physical + commit) behind a 1s timer to reduce repeated sampling overhead.
• Extend BugSplat crash attributes with additional memory telemetry.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 10 comments.

Show a summary per file

File	Description
indra/newview/llviewertexture.h	Declares Windows-only commit-memory accessor for texture system.
indra/newview/llviewertexture.cpp	Adds cached sampling and incorporates commit availability into low/critical memory checks on Windows.
indra/newview/llappviewerwin32.cpp	Adds crash-time memory stats to BugSplat attributes.
indra/llcommon/llsys.h	Changes LLMemoryInfo available-memory API to update internal cached stats rather than returning via out-param.
indra/llcommon/llsys.cpp	Implements new memory stat update path; populates LLMemory-backed cached fields on Windows and other platforms.
indra/llcommon/llmemory.h	Adds Windows-only commit-availability accessor and allows LLMemoryInfo to update LLMemory’s cached stats.
indra/llcommon/llmemory.cpp	Adapts updateMemoryInfo() to new LLMemoryInfo API and exposes commit availability getter.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

Comments suppressed due to low confidence (1)

indra/llcommon/llsys.h:142

• The comment says all stats-map values are in kilobytes, but this PR changes Windows virtual-memory values to be reported in megabytes (e.g., "Total Virtual MB"). Please update the comment to avoid misleading consumers.

    // Retrieve a map of memory statistics. The keys of the map are platform-
    // dependent. The values are in kilobytes to try to avoid integer overflow.
    LLSD getStatsMap() const;

[Copilot]

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 4 comments.

[Copilot]

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

[Copilot]

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 5 comments.

[Copilot]

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 5 comments.

[Copilot]

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 3 comments.