[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