[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