This is an archive of the discontinued LLVM Phabricator instance.

[scudo] Check support for pvalloc/valloc tests.
ClosedPublic

Authored by cferris on Jun 12 2023, 4:24 PM.

Details

Summary

Some platforms do not support pvalloc/valloc so add checks so
that these tests are only run on the appropriate platforms.

Diff Detail

Event Timeline

cferris created this revision.Jun 12 2023, 4:24 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 12 2023, 4:24 PM
cferris requested review of this revision.Jun 12 2023, 4:24 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 12 2023, 4:24 PM
Herald added a subscriber: Restricted Project. · View Herald Transcript
cferris updated this revision to Diff 530700.Jun 12 2023, 4:26 PM

Fix location of test file.

Just curious, why didn't this cause problem on Android before?

compiler-rt/lib/scudo/standalone/tests/wrappers_c_test.cpp
31

Hope this make it easier to read

Chia-hungDuan accepted this revision.Jun 12 2023, 4:36 PM
This revision is now accepted and ready to land.Jun 12 2023, 4:36 PM
cferris updated this revision to Diff 530706.Jun 12 2023, 4:41 PM

More readable.

cferris marked an inline comment as done.Jun 12 2023, 4:42 PM

We aren't compiling this in Android yet. I think there was a problem and we just never went back and fixed it.

We aren't compiling this in Android yet. I think there was a problem and we just never went back and fixed it.

Got it! Thanks for the explanation

This revision was automatically updated to reflect the committed changes.