Page MenuHomePhabricator

[CUDA] Minor cuda-related driver fixes.
ClosedPublic

Authored by tra on Sep 15 2015, 5:03 PM.

Details

Summary
  • Only the last of the --cuda-host-only/--cuda-device-only options has effect.
  • Added test cases.
  • Fixed printing of empty action lists for -ccc-print-phases

Diff Detail

Repository
rL LLVM

Event Timeline

tra updated this revision to Diff 34855.Sep 15 2015, 5:03 PM
tra retitled this revision from to [CUDA] Minor cuda-related driver fixes..
tra updated this object.
tra added a reviewer: echristo.
tra added a subscriber: cfe-commits.
echristo accepted this revision.Sep 15 2015, 5:06 PM
echristo edited edge metadata.

Yuck. No better ideas off the top of my head other than diagnosing and erroring though.

(Is that a possibility? :)

-eric

This revision is now accepted and ready to land.Sep 15 2015, 5:06 PM
tra added a comment.Sep 15 2015, 5:21 PM

Yuck. No better ideas off the top of my head other than diagnosing and erroring though.

(Is that a possibility? :)

I need to be able to override preceding options, so no error is intentional.

Sure. Seems reasonable.

Thanks!

-eric

This revision was automatically updated to reflect the committed changes.