This revision was automatically updated to reflect the committed changes.
Closed by commit rG529f4ed401ea: Improve __builtin_constant_p lowering
(authored by joerg).
Changed prior to commit:
https://reviews.llvm.org/D67638?vs=220398&id=224796#toc
Repository:
rG LLVM Github Monorepo
CHANGES
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
OK, this is fine by me if you're confident this doesn't degrade the generated
code at -O0.
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D67638/new/
https://re
joerg added a comment.
Ping.
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D67638/new/
https://reviews.llvm.org/D67638
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/lis
joerg created this revision.
joerg added a reviewer: rsmith.
Herald added subscribers: kristina, kbarton, nemanjai.
Herald added a project: clang.
Herald added a subscriber: wuzish.
__builtin_constant_p used to be short-cut evaluated to false when building with
-O0. This is undesirable as it mean