Author: deadalnix
Date: Thu Sep 8 23:42:49 2016
New Revision: 281018
URL: http://llvm.org/viewvc/llvm-project?rev=281018=rev
Log:
Update clang for D21514. NFC
Summary: As per title.
Reviewers: ahatanak, bkramer, whitequark, mehdi_amini, void
Subscribers: cfe-commits
Differential Revision:
This revision was automatically updated to reflect the committed changes.
Closed by commit rL281018: Update clang for D21514. NFC (authored by deadalnix).
Changed prior to commit:
https://reviews.llvm.org/D21515?vs=62791=70783#toc
Repository:
rL LLVM
https://reviews.llvm.org/D21515
Files:
deadalnix updated this revision to Diff 62791.
deadalnix added a comment.
rebase/ping
http://reviews.llvm.org/D21515
Files:
lib/CodeGen/CGExpr.cpp
Index: lib/CodeGen/CGExpr.cpp
===
--- lib/CodeGen/CGExpr.cpp
+++
deadalnix created this revision.
deadalnix added reviewers: ahatanak, bkramer, whitequark, mehdi_amini, void.
deadalnix added a subscriber: cfe-commits.
As per title.
http://reviews.llvm.org/D21515
Files:
lib/CodeGen/CGExpr.cpp
Index: lib/CodeGen/CGExpr.cpp
deadalnix requested changes to this revision.
This revision now requires changes to proceed.
Comment at: include/llvm-c/Core.h:604
@@ -603,3 +603,3 @@
*/
-LLVMBool LLVMPrintModuleToFile(LLVMModuleRef M, const char *Filename,
+bool LLVMPrintModuleToFile(LLVMModuleRef M, const
deadalnix added a comment.
In http://reviews.llvm.org/D19299#406517, @spatel wrote:
> Rereading your question, I'm now wondering if you are asking if we can get
> rid of the source level builtin_unpredictable() ? I had not considered that,
> but I think that is also possible if we add a flag
deadalnix added a subscriber: deadalnix.
Comment at: lib/CodeGen/CGStmt.cpp:1560-1588
@@ -1560,1 +1559,31 @@
+MDHelper.createUnpredictable());
+ } else if (FD->getBuiltinID() == Builtin::BI__builtin_expect) {
+
+// FIXME:
