This is an archive of the discontinued LLVM Phabricator instance.

[AIX][test-suite] Create a new reference output on AIX to account for differences in random()
ClosedPublic

Authored by amyk on May 24 2021, 9:29 PM.

Details

Summary

The random() function on AIX appears to be implemented differently and produces
slightly different output when compared to the implementation found on Linux.

This patch creates a new reference output on AIX for the following benchmarks:

Prolangs-C++/city
Ptrdist/ft

Diff Detail

Repository
rT test-suite

Event Timeline

amyk created this revision.May 24 2021, 9:29 PM
amyk requested review of this revision.May 24 2021, 9:29 PM
amyk added a subscriber: ZarkoCA.Aug 16 2021, 7:25 AM

Gentle ping.
@hubert.reinterpretcast @ZarkoCA @stevewan @daltenty Did any of you have any comments regarding this change?

amyk added a comment.Sep 14 2021, 10:01 AM

Gentle ping.

The fix looks good to me in general, though I'll wait the author to re-assess the work split between this and D102409.

stevewan accepted this revision as: stevewan.EditedSep 15 2021, 6:33 PM

LGTM. FWIW, be mindful of the commit sequence, either this or D102409 will perhaps need to be rebased on the other.

This revision is now accepted and ready to land.Sep 15 2021, 6:33 PM