[PATCH] D131343: [RISC-V][HWASAN] Add support for lowering HWASAN intrinsic for RISC-V

Alexey Baturo via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 16 11:41:04 PDT 2022


smd added a comment.

@vitalybuka

> llvm/test/CodeGen/RISCV/hwasan-check-memaccess.ll ?
> the test essentially validates LowerHWASAN_CHECK_MEMACCESS, and does not relies on the rest of D131344 <https://reviews.llvm.org/D131344>
> So the test needs to be moved from D131344 <https://reviews.llvm.org/D131344> into this patch.

I believe the same applies to D131575 <https://reviews.llvm.org/D131575> and basic.ll, where changes to instrumentMemAccessInline are tested? 
If so, would you mind me moving hwasan-check-memaccess.ll to this patch and the rest of .ll tests to D131575 <https://reviews.llvm.org/D131575>?
Thanks


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131343



More information about the llvm-commits mailing list