[PATCH] D40147: [MIPS] Handle cross-mode (regular <-> microMIPS) jumps
Rui Ueyama via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Nov 26 16:37:28 PST 2018
ruiu added a comment.
Adding a new parameter to `relocateOne` doesn't look beautiful, but that makes me think whether you are editing the right place of the code. `relocateOne` is supposed to apply a single relocation based on a value passed to the function. No complex logic should be in the function. Passing one more parameter so that you can do more complex computation in the function may not be a good idea.
Repository:
rLLD LLVM Linker
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D40147/new/
https://reviews.llvm.org/D40147
More information about the llvm-commits
mailing list