[PATCH] D116161: [Clang] Add an overload for emitUnaryBuiltin.

[Florian Hahn via Phabricator via cfe-commits]

fhahn added a comment.

In D116161#3209286 <https://reviews.llvm.org/D116161#3209286>, @junaire wrote:

>   35:  %0 = load float, float* %f1.addr, align 4 
>   36:  %1 = load float, float* %f1.addr, align 4 
>   37:  %elt.abs = call float @llvm.fabs.f32(float %1) 

It looks like the argument expression is evaluated twice. Did you remove the `    Value *Op0 = EmitScalarExpr(E->getArg(0));` calls?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D116161/new/

https://reviews.llvm.org/D116161