[PATCH] D45808: [OpenCL] Add 'denorms-are-zero' function attribute

2018-04-20 Thread Alexey Sotkin via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL330404: [OpenCL] Add denorms-are-zero function attribute (authored by AlexeySotkin, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM https://reviews.llvm.org/D45808 Files:

[PATCH] D45808: [OpenCL] Add 'denorms-are-zero' function attribute

2018-04-19 Thread Kristina Bessonova via Phabricator via cfe-commits
krisb updated this revision to Diff 143072. krisb added a comment. Applied the comments Repository: rC Clang https://reviews.llvm.org/D45808 Files: lib/CodeGen/CGCall.cpp test/CodeGenOpenCL/denorms-are-zero.cl Index: test/CodeGenOpenCL/denorms-are-zero.cl

[PATCH] D45808: [OpenCL] Add 'denorms-are-zero' function attribute

2018-04-19 Thread Yaxun Liu via Phabricator via cfe-commits
yaxunl accepted this revision. yaxunl added a comment. This revision is now accepted and ready to land. LGTM. Thanks! Repository: rC Clang https://reviews.llvm.org/D45808 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D45808: [OpenCL] Add 'denorms-are-zero' function attribute

2018-04-19 Thread Anastasia Stulova via Phabricator via cfe-commits
Anastasia added inline comments. Comment at: test/CodeGenOpenCL/denorms-are-zero.cl:9 +// CHECK-LABEL: define void @f() +// CHECK: attributes #{{[0-9]*}} = {{{[^}]*}} "denorms-are-zero"="true" Could we add a check for the false case too?

[PATCH] D45808: [OpenCL] Add 'denorms-are-zero' function attribute

2018-04-19 Thread Kristina Bessonova via Phabricator via cfe-commits
krisb created this revision. krisb added reviewers: Anastasia, yaxunl. Herald added a subscriber: cfe-commits. Generate attribute 'denorms-are-zero'='true' if '-cl-denorms-are-zero' compile option was specified and 'denorms-are-zero'='false' otherwise. Repository: rC Clang