Fix error when compile tvm with latest llvm14git

[Joejiong]

Thanks for contributing to TVM! Please refer to guideline https://tvm.apache.org/docs/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from Reviewers by @ them in the pull request thread.

Fix two compilation error when compile tvm with latest llvm when version >=12;

• llvm::Intrinsic::getName doesn't have function overload for two arguments; (https://llvm.org/doxygen/Intrinsics_8h.html)
• llvm::sys::fs::F_None(doesn't exists any more) --> llvm::sys::fs::OF_None; (https://llvm.org/doxygen/FileSystem_8h.html)

[junrushao]

Thanks for the fix! I am not the expert of the LLVM backend, so CC’ing related experts @kparzysz-quic @FrozenGene @zhiics

[Joejiong]

Thanks for the fix! I am not the expert of the LLVM backend, so CC’ing related experts @kparzysz-quic @FrozenGene @zhiics

your CI got some problem with latest llvm, clearly, the following two things have changed for a while.

• llvm::Intrinsic::getName doesn't have function overload for two arguments; (https://llvm.org/doxygen/Intrinsics_8h.html)
• llvm::sys::fs::F_None(doesn't exists any more) --> llvm::sys::fs::OF_None; (https://llvm.org/doxygen/FileSystem_8h.html)

the problem happens when you build your own llvm and change the cmake.config pointing to the /latest/path/to/llvm-config, then you will get several compilation errors.

[junrushao]

To keep backward compatibility with earlier LLVM distributions, please use macros to guard the code path. Thanks :-)

[Joejiong]

earlier

thx, ok, sorry didn't see the surrounding examples, hahaha~

[tqchen]

Thank you @Joejiong !

[deeply]

Seems that llvm-13 has already changed the signature of llvm::Intrinsic::getName in the latest rc2. Maybe we need to update the guard macro to #if TVM_LLVM_VERSION <= 120 for calling llvm::Intrinsic::getName? 😄