Re: [PATCH] D12458: [clang] add __builtin_unpredictable and convert to metadata

2015-09-02 Thread Sanjay Patel via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL246699: add __builtin_unpredictable and convert to metadata (authored by spatel). Changed prior to commit: http://reviews.llvm.org/D12458?vs=33491&id=33850#toc Repository: rL LLVM http://reviews.llv

Re: [PATCH] D12458: [clang] add __builtin_unpredictable and convert to metadata

2015-08-31 Thread Krzysztof Parzyszek via cfe-commits
kparzysz added a comment. LGTM Comment at: lib/CodeGen/CodeGenFunction.cpp:1238 @@ -1217,3 +1237,3 @@ } - Builder.CreateCondBr(CondV, TrueBlock, FalseBlock, Weights); + Builder.CreateCondBr(CondV, TrueBlock, FalseBlock, Weights, Unpredictable); } spatel wr

Re: [PATCH] D12458: [clang] add __builtin_unpredictable and convert to metadata

2015-08-31 Thread Sanjay Patel via cfe-commits
spatel added inline comments. Comment at: lib/CodeGen/CodeGenFunction.cpp:1238 @@ -1217,3 +1237,3 @@ } - Builder.CreateCondBr(CondV, TrueBlock, FalseBlock, Weights); + Builder.CreateCondBr(CondV, TrueBlock, FalseBlock, Weights, Unpredictable); } spatel wrote

Re: [PATCH] D12458: [clang] add __builtin_unpredictable and convert to metadata

2015-08-31 Thread Krzysztof Parzyszek via cfe-commits
kparzysz added inline comments. Comment at: lib/CodeGen/CodeGenFunction.cpp:1238 @@ -1217,3 +1237,3 @@ } - Builder.CreateCondBr(CondV, TrueBlock, FalseBlock, Weights); + Builder.CreateCondBr(CondV, TrueBlock, FalseBlock, Weights, Unpredictable); } Is the def

[PATCH] D12458: [clang] add __builtin_unpredictable and convert to metadata

2015-08-28 Thread Sanjay Patel via cfe-commits
spatel created this revision. spatel added reviewers: hfinkel, kbsmith1, kparzysz, reames, silvas. spatel added a subscriber: cfe-commits. spatel added a dependency: D12341: add unpredictable metadata type for control flow. This patch depends on D12341 ('unpredictable' metadata type for LLVM).