This is an archive of the discontinued LLVM Phabricator instance.

extra test modifications for D34158
ClosedPublic

Authored by mibintc on Jun 26 2017, 8:12 AM.

Details

Summary

When adding support for D34158 which changes preprocessed output, I needed to make tiny test corrections for these. Adding the option -ffreestanding suppresses the new behavior, and that's the change I made to fix the tests.

Diff Detail

Repository
rL LLVM

Event Timeline

mibintc created this revision.Jun 26 2017, 8:12 AM
mibintc updated this revision to Diff 105852.Jul 10 2017, 8:11 AM

With the latest proposed fix for D34158, a few more test corrections are needed. The correction consists of suppressing the new preprocessor behavior. D34158 preincludes the file <stdc-predef.h> if the file is available.

mibintc planned changes to this revision.Jul 10 2017, 12:42 PM

I need to redo the test changes. Will resubmit the diff.

mibintc updated this revision to Diff 105906.Jul 10 2017, 1:12 PM
mibintc edited the summary of this revision. (Show Details)

This patch corresponds to these modifications to clang: https://reviews.llvm.org/D34158?id=105904

mibintc updated this revision to Diff 110059.Aug 7 2017, 12:55 PM

Updating this patch to latest revision of tools/extra

mibintc updated this revision to Diff 114887.Sep 12 2017, 1:06 PM

Updating to latest version. No change other than that.

mibintc updated this revision to Diff 120584.Oct 27 2017, 6:18 AM

Updated the test diff to correspond to the D34158 patch uploaded today.

This revision was automatically updated to reflect the committed changes.