This is an archive of the discontinued LLVM Phabricator instance.

Add more f optimizations to ignore for compatibility
ClosedPublic

Authored by bubbles231 on Jul 17 2014, 10:05 AM.

Details

Reviewers
rafael
rnk
Summary

Added flags that should be ignored for compatibility. These flags are in the clang_ignored_gcc_optimization_f_Group so they will display a warning.

Diff Detail

Event Timeline

bubbles231 retitled this revision from to Add more f optimizations to ignore for compatibility .
bubbles231 updated this object.
bubbles231 edited the test plan for this revision. (Show Details)
bubbles231 added a subscriber: Unknown Object (MLST).
bubbles231 updated this revision to Diff 11640.Jul 18 2014, 6:22 AM

Updated the patch for new svn revision, add test for warnings, and prepare it for being a dependency of: http://reviews.llvm.org/D4570.

bubbles231 updated this revision to Diff 11645.EditedJul 18 2014, 9:36 AM

Move a lot of flags from http://reviews.llvm.org/D4570 and move them to this revision, make corrections, and add tests.

bubbles231 updated this object.Jul 18 2014, 9:48 AM
rnk edited edge metadata.Aug 4 2014, 2:25 PM

Some of these flags are a bit functional, like -flto, -fwhole-program, and -fuse-linker-plugin.

Rafael, do you have any thoughts about this list, since you added the "error on unknown options" logic?

rnk accepted this revision.Aug 5 2014, 11:05 AM
rnk edited edge metadata.

Looks good to me as well.

This revision is now accepted and ready to land.Aug 5 2014, 11:05 AM