[PATCH] AArch64: look up BuiltinID before calling EmitAArch64ScalarBuiltinExpr
apazos at codeaurora.org
Tue Feb 18 12:56:27 PST 2014
From: Tim Northover [mailto:t.p.northover at gmail.com]
Sent: Saturday, February 15, 2014 12:52 AM
To: Ana Pazos
Cc: reviews+D2764+public+ce3378023644676a at llvm-reviews.chandlerc.com; cfe-commits at cs.uiuc.edu
Subject: Re: [PATCH] AArch64: look up BuiltinID before calling EmitAArch64ScalarBuiltinExpr
Thanks for taking a look at the patch.
> Can you please explain again why a table is used for the scalar intrinsics and not the other intrinisics? The other intrinsics also have long case statements.
Basically because I had to start somewhere. The scalar intrinsics are the natural place because the switch statement is already phrased as "set a few parameters & break", we just have to delete it completely.
I definitely agree the idea is applicable to the SIMD case as well, and I've actually done quite a bit of work in that direction; but I wanted to make sure everyone else thought the approach was sound before pressing on. Incremental patches and all that.
More information about the cfe-commits