This is an archive of the discontinued LLVM Phabricator instance.

Add a few dllimport/dllexport tests. NFC
ClosedPublic

Authored by wristow on Mar 1 2016, 10:53 AM.

Details

Summary

This change just adds tests for some corner cases of dllimport/dllexport,
primarily for some static methods.

We plan to enable dllimport/dllexport support for the PS4, and these
additional tests are for points we previously were testing internally.

Can someone review, and if OK then commit this patch for me, please?

-Warren Ristow
SN Systems - Sony Computer Entertainment Group

Diff Detail

Repository
rL LLVM

Event Timeline

wristow updated this revision to Diff 49510.Mar 1 2016, 10:53 AM
wristow retitled this revision from to Add a few dllimport/dllexport tests. NFC.
wristow updated this object.
wristow added reviewers: cfe-commits, rnk, majnemer.
silvas added a subscriber: silvas.Mar 1 2016, 12:03 PM

Random question:

We plan to enable dllimport/dllexport support for the PS4

How are you planning on codegenning them? The same way as we do in our private branch?

Random question:

We plan to enable dllimport/dllexport support for the PS4

How are you planning on codegenning them? The same way as we do in our private branch?

Yes, that's the intention.

This revision was automatically updated to reflect the committed changes.