This is an archive of the discontinued LLVM Phabricator instance.

Do not duplicate identical stack safety test for RISCV
ClosedPublic

Authored by fmayer on Aug 29 2023, 4:55 PM.

Diff Detail

Event Timeline

fmayer created this revision.Aug 29 2023, 4:55 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 29 2023, 4:55 PM
fmayer requested review of this revision.Aug 29 2023, 4:55 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 29 2023, 4:55 PM
fmayer added a reviewer: smd.Aug 29 2023, 5:02 PM
vitalybuka accepted this revision.Aug 29 2023, 8:30 PM
This revision is now accepted and ready to land.Aug 29 2023, 8:30 PM
jrtc27 added inline comments.Aug 29 2023, 8:36 PM
llvm/test/Instrumentation/HWAddressSanitizer/stack-safety-analysis.ll
11–12

This is a bit dodgy as this isn't a data layout used by RISC-V, but hard to avoid without D141060 unless you want to duplicate the test or put the data layout as an argument to opt

fmayer added inline comments.Aug 29 2023, 9:12 PM
llvm/test/Instrumentation/HWAddressSanitizer/stack-safety-analysis.ll
11–12

For what it's worth, that is also the data layout that is used in the deleted llvm/test/Instrumentation/HWAddressSanitizer/RISCV/stack-safety-analysis.ll.

This revision was automatically updated to reflect the committed changes.