This is an archive of the discontinued LLVM Phabricator instance.

[Fuzzer] Don't hardcode target architecture for Fuzzer tests
ClosedPublic

Authored by phosek on Jun 14 2018, 8:59 PM.

Diff Detail

Event Timeline

phosek created this revision.Jun 14 2018, 8:59 PM
Herald added subscribers: Restricted Project, llvm-commits, mgorny. · View Herald TranscriptJun 14 2018, 8:59 PM

r333241 which introduced this change broke Fuchsia toolchain build because we build compiler-rt on/for aarch64. The build log can be seen here: https://logs.chromium.org/v/?s=fuchsia%2Fbuildbucket%2Fcr-buildbucket.appspot.com%2F8943696681206442864%2F%2B%2Fsteps%2Fclang%2F0%2Fsteps%2Fbuild%2F0%2Fstdout

morehouse accepted this revision.Jun 15 2018, 9:15 AM

Seems reasonable to me.

@george.karpenkov: Any comments on this?

This revision is now accepted and ready to land.Jun 15 2018, 9:15 AM
george.karpenkov accepted this revision.Jun 15 2018, 9:27 AM

Yeah, totally makes sense! Apologies for breaking this.

vitalybuka accepted this revision.Jun 15 2018, 11:23 AM
This revision was automatically updated to reflect the committed changes.