This is an archive of the discontinued LLVM Phabricator instance.

[Patch] Temporarily revert test to prepare for LLVM -fprofile-instr-generate= fix
AbandonedPublic

Authored by tejohnson on May 12 2015, 1:52 PM.

Details

Reviewers
echristo
bogner
Summary

I apparently forgot to update the clang and llvm patches before
they were committed for me with the new name of the compiler-rt library
supporting -fprofile-instr-generate=, which was changed during the review
process. This wasn't caught by this test as it doesn't link and is consistent
with the library call being inserted by llvm.

I want to temporarily revert this test so that it doesn't break when I make
the llvm change to call the correct library routine
(__llvm_profile_override_default_filename), then will send a patch to add itx
back with the correct name.

Tested via ninja clang-test

Diff Detail

Event Timeline

tejohnson updated this revision to Diff 25617.May 12 2015, 1:52 PM
tejohnson retitled this revision from to [Patch] Temporarily revert test to prepare for LLVM -fprofile-instr-generate= fix.
tejohnson updated this object.
tejohnson edited the test plan for this revision. (Show Details)
tejohnson added reviewers: echristo, bogner.
tejohnson added a subscriber: Unknown Object (MLST).
echristo edited edge metadata.May 12 2015, 1:57 PM

Should just commit quickly. No worries about a breakage of a few minutes.

tejohnson abandoned this revision.Aug 12 2015, 7:43 AM

Obsolete, didn't need to do this.