[PATCH] D140528: [MSan] Enable MSAN for loongarch64

[Limin Zhang via Phabricator via llvm-commits]

Ami-zhang added inline comments.


================
Comment at: compiler-rt/lib/msan/msan_allocator.cpp:82
+#elif defined(__loongarch_lp64)
+static const uptr kAllocatorSpace = 0x700000000000ULL;
+static const uptr kMaxAllowedMallocSize = 8UL << 30;
----------------
MaskRay wrote:
> namespace-scope const is automatically internal linkage, so you can omit `static`
Ok, I have  omitted `static` in the namespace-scope const.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D140528/new/

https://reviews.llvm.org/D140528