[PATCH] D49673: [AArch64] Add Tiny Code Model for AArch64

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 15 04:21:57 PDT 2018


dmgreen added a comment.

In https://reviews.llvm.org/D49673#1199590, @dmgreen wrote:

> Hmm.. So I was expecting this would keep working the same as small model. I believe that would be less efficient but functionally correct (providing you are not un-aligning page boundaries). In adding some tests I can see that there are some code differences though (because it goes through LOADgot). I think the example in arm64-tls-execs is correct, and the other tests here produce the same code (as does your example from above). I've not tested this very well though, and I'm not sure what else would be required. As far as I can tell it should be OK, but producing an error for tiny+tls, same as large+tls, is an option.


I've been testing out some examples with tls + tiny [+ pic]. As far as I can tell from the code I've ran, everything is working as expected.


https://reviews.llvm.org/D49673





More information about the llvm-commits mailing list