MichaReiser / llvm-node

LLVM 9.0+ Node Bindings
MIT License
183 stars 29 forks source link

Include target triple in lookupTarget error message #54

Closed emlai closed 6 years ago

emlai commented 6 years ago

Previously the error message was e.g. "Failed to lookup target: No available targets are compatible with this triple." I think this will be more helpful when it mentions what the target triple was.

Another possibility would be to pass the error message from LLVM unchanged, and let the user build a better error message themselves.