This is an archive of the discontinued LLVM Phabricator instance.

[scudo][standalone] Fix tests makefile
ClosedPublic

Authored by cryptoad on Feb 28 2019, 2:04 PM.

Details

Summary

A missing STATIC entailed some annoying to debug failures wrt 32 vs 64
binaries. Additionally I noticed I was using the wrong variable (the Scudo
one as opposed to the Scudo Standalone one).

See https://reviews.llvm.org/D58184#1412417 and below for discussion.

Diff Detail

Event Timeline

cryptoad created this revision.Feb 28 2019, 2:04 PM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptFeb 28 2019, 2:04 PM
Herald added subscribers: Restricted Project, delcypher, mgorny. · View Herald Transcript
vitalybuka accepted this revision.Feb 28 2019, 4:38 PM
This revision is now accepted and ready to land.Feb 28 2019, 4:38 PM
brzycki accepted this revision.Mar 1 2019, 7:36 AM

Thank you for helping debug this @cryptoad .

This revision was automatically updated to reflect the committed changes.