[PATCH] D16674: [mips] Expansion of SC[D] and LL[D] for R6

Daniel Sanders via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 7 05:38:41 PDT 2016


dsanders added a comment.

One higher level comment I'd like to add to this is that I'm a somewhat reluctant to add support for the out-of-range offset macros that GAS supports. I currently think it has a very high cost in terms of implementation effort, testing effort, and maintenance for very little benefit. I'm not aware of any code that uses this feature intentionally and having IAS reject accidental usage of it can be beneficial in the case of ll/sc loops to avoid unnecessary instructions inside the loop.


http://reviews.llvm.org/D16674





More information about the llvm-commits mailing list