[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