Mention additional requirements

[mirh]

Shouldn't you mention you need OpenCL (I guess your APP SDK runtime libraries)?

EDIT: and Microsoft Foundation Classes for at least one of the samples

[GeoffPark]

To compile, you do need OpenCL headers. The OpenCL runtime is included with Radeon drivers. Note that TAN library also has a fully functional, though slower, CPU mode which does not require OpenCL runtime or a GPU.

[mirh]

To compile, you do need OpenCL headers.

Yes, I was pointing out that this requirement is not mentioned in your readme prerequisites.

[GeoffPark]

Thanks for pointing that out. We will correct the readme in next update.

-Geoff

From: mirh [mailto:notifications@github.com]
Sent: Friday, August 19, 2016 5:35 PM
To: GPUOpen-LibrariesAndSDKs/TAN TAN@noreply.github.com
Cc: Park, Geoffrey Geoffrey.Park@amd.com; Comment comment@noreply.github.com
Subject: Re: [GPUOpen-LibrariesAndSDKs/TAN] Mention additional requirements (#1)

To compile, you do need OpenCL headers.

Yes, I was pointing out that this requirement is not mentioned in your readme prerequisites.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com//issues/1#issuecomment-241141391, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AP4h_LLzd6XWRKWO7fdHpYOOjW3XEPb5ks5qhiGQgaJpZM4JnyZb.