vitalybuka added inline comments.

================
Comment at: compiler-rt/lib/hwasan/hwasan_mapping.h:57
 
 extern uptr kAliasRegionStart;
 
----------------
Maybe the following to minimize code which compiles only on limited set of 
platforms?

```
inline uptr GetShadowOffset() {
  return SANITIZER_FUCHSIA ? 0 : __hwasan_shadow_memory_dynamic_address;
}
```


================
Comment at: compiler-rt/lib/hwasan/hwasan_mapping.h:59
 
 inline uptr MemToShadow(uptr untagged_addr) {
+  return (untagged_addr >> kShadowScale) + SHADOW_OFFSET
----------------
> clang-tidy: error: expected ';' after return statement 
> [clang-diagnostic-error]
please fix


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104275

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to