[PATCH] D34409: Use 64bit jump table with large code model on 64bit
Yichao Yu via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jun 21 05:10:50 PDT 2017
yuyichao added a comment.
> If the jump table is writable
FWIW, that also sounds like a hack and a (minor?) security issue.
> It would be really better to work on the real fix and not add more hacks...
And what exactly do you mean by "the real fix". I believe this is a reasonable generic fallback before an optimization is implemented for a particular arch and given that this is exactly what GCC does on x86-64 I think this is (one of) the correct solution on x86-64 too.
On AArch64, GCC throws an error, which I think is much better than silently generating the wrong code....
https://reviews.llvm.org/D34409
More information about the llvm-commits
mailing list