[PATCH] Fix ARM AAPCS regression caused by r211898
Renato Golin
renato.golin at linaro.org
Thu Jul 17 12:36:06 PDT 2014
Hi Oliver,
ByVal is normally a big deal for performance when structures are passed as arguments, and I don't think we should disable it like that. I don't have benchmark numbers to tell if it really is worth keep it, maybe you can disable it and run some benchmarks.
All in all, this commit looks ok and if no one is against it, feel free to commit. We can leave the ByVal discussion for later.
cheers,
--renato
http://reviews.llvm.org/D4541
More information about the cfe-commits
mailing list