[PATCH] D105465: [RuntimeDyld] Added support for relocation of indirect functions

Lang Hames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 31 08:38:28 PDT 2022


lhames added a comment.

> The only failing "test" is clang-format. Should I reformat the complete files? Because my additions are formatted correctly, but the rest of the file is not.

Only your additions should be reformatted. Reformatting the rest of the files will obscure the commit history. It's better to let them be updated to the correct format over time as patches come in. (Though my hope is that we can kill RuntimeDyld in the not too distant future and just move everyone over to JITLink)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D105465/new/

https://reviews.llvm.org/D105465



More information about the llvm-commits mailing list