search

ROCm / HIPIFY

HIPIFY: Convert CUDA to Portable C++ Code
https://rocm.docs.amd.com/projects/HIPIFY/en/latest/
MIT License
503 stars 70 forks source link

[HIPIFY][perl] hipify-perl should support --help and --version options #290

Closed boegel closed 3 years ago

boegel commented 3 years ago

It would be useful if hipify-perl would support a --help option, to give information about basic usage, and to print all supported options (like --inplace).

$ hipify-perl --help
Unknown option: help

Likewise, supporting --version would be helpful too.

emankov commented 3 years ago

What do you want from the script under the --version option?

boegel commented 3 years ago

@emankov Well, just print the version number? Something like:

$ hipify-perl --version
4.0.0
emankov commented 3 years ago

So, do you need ROCm version?

boegel commented 3 years ago

I don't need anything in particular, I mostly would like that --version is supported, since it's a pretty common option for CLI tools. If HIPIFY doesn't have it's own version number, then printing the ROCm version makes sense I guess.

emankov commented 3 years ago

hipify-clang being clang tooling based tool prints LLVM version against which it is built. For hipify-perl let it be the HIP version.

emankov commented 3 years ago

Closing it as implemented by #291 and #292.