This is an archive of the discontinued LLVM Phabricator instance.

[CUDA] Handle -O options (more) correctly.
ClosedPublic

Authored by jlebar on Jan 18 2016, 4:19 PM.

Details

Summary

Previously we'd crash the driver if you passed -O0. Now we try to
handle all of clang's various optimization flags in a sane way.

Diff Detail

Event Timeline

jlebar updated this revision to Diff 45219.Jan 18 2016, 4:19 PM
jlebar retitled this revision from to [CUDA] Handle -O options (more) correctly..
jlebar updated this object.
jlebar added a reviewer: tra.
jlebar added subscribers: jhen, echristo, cfe-commits.

At a quick glance Os and Oz should probably map to O2.

jlebar updated this revision to Diff 45222.Jan 18 2016, 5:40 PM

Map -Os, -Oz to -O2, rather than -O3.

jlebar updated this revision to Diff 45223.Jan 18 2016, 5:41 PM

Update comment

tra accepted this revision.Jan 19 2016, 10:16 AM
tra edited edge metadata.

LGTM.

This revision is now accepted and ready to land.Jan 19 2016, 10:16 AM
This revision was automatically updated to reflect the committed changes.