jina.nahias updated this revision to Diff 114765.
https://reviews.llvm.org/D37668
Files:
include/clang/Basic/BuiltinsX86.def
include/clang/Basic/BuiltinsX86_64.def
lib/Headers/avx512bwintrin.h
lib/Headers/avx512fintrin.h
lib/Headers/avx512vlbwintrin.h
lib/Headers/avx512vlintrin.h
te
kristof.beyls added a comment.
Still LGTM; please commit.
https://reviews.llvm.org/D34878
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
erichkeane added a comment.
You seem to have had a hard time with the diff tool too... there is an extra
file here that needs to be removed.
Comment at: lib/Sema/SemaDeclCXX.cpp:2390
+ Uuid && Uuid->getGuid() =="---C000-0046" &&
+ dyn_ca
anemet added inline comments.
Comment at: lib/CodeGen/CodeGenAction.cpp:882-883
BEConsumer = Result.get();
-
+ VMContext->setDiagnosticHandler(llvm::make_unique(
+ CI.getCodeGenOpts(), Result.get()));
// Enable generating macro debug info only when debug info is not d
anemet accepted this revision.
anemet added a comment.
This revision is now accepted and ready to land.
LGTM with the nits below. Thanks!
Comment at: include/llvm/IR/DiagnosticHandler.h:1
+//===- DiagnosticHandler.cpp - DiagnosticHandler class for LLVM -*- C++
-*-===//
+//
--
ahatanak updated this revision to Diff 114753.
ahatanak added a comment.
Address review comments.
- Detect invalid references to parameters or local variables by default
arguments in tryCaptureVariable. Before parsing or instantiating the default
argument expression, the enclosing DeclContext i
martell created this revision.
Alias `-static-compiler-rt` and `static-libc++` to their gcc counterparts.
Currently invoking `-static-libgcc` or `-static-libstdc++` will ensure only use
of static libs.
In future if we want to do more accurate handling with better behaviour with
the linker we can
This revision was automatically updated to reflect the committed changes.
Closed by commit rL312989: [Driver] Fuchsia targets default to
-fasynchronous-unwind-tables (authored by phosek).
Changed prior to commit:
https://reviews.llvm.org/D37723?vs=114737&id=114746#toc
Repository:
rL LLVM
ht
Author: phosek
Date: Mon Sep 11 18:28:37 2017
New Revision: 312989
URL: http://llvm.org/viewvc/llvm-project?rev=312989&view=rev
Log:
[Driver] Fuchsia targets default to -fasynchronous-unwind-tables
This regressed for x86-64 in r307856 because it's no longer inherited
from Generic_GCC. We'd never
Author: vlad.tsyrklevich
Date: Mon Sep 11 17:21:17 2017
New Revision: 312986
URL: http://llvm.org/viewvc/llvm-project?rev=312986&view=rev
Log:
Fix broken links to the Itanium CXX ABI
Modified:
cfe/trunk/docs/ControlFlowIntegrityDesign.rst
cfe/trunk/docs/Toolchain.rst
cfe/trunk/lib/AST
phosek accepted this revision.
phosek added a comment.
This revision is now accepted and ready to land.
LGTM
Repository:
rL LLVM
https://reviews.llvm.org/D37723
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin
mcgrathr created this revision.
Herald added subscribers: kristof.beyls, aemerson.
This regressed for x86-64 in r307856 because it's no longer inherited
from Generic_GCC. We'd never noticed that it was missing other
targets (i.e. aarch64), but Fuchsia is uniform across all machines.
Repository:
martell accepted this revision.
martell added a comment.
LGMT
https://reviews.llvm.org/D37530
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
rsmith added a comment.
If this is just supposed to be an experiment to get feedback on the feature,
then I don't think we should be treating it as a different attribute syntax at
all. Rather, I think we just want to permit C++11 attributes to be parsed in
other language modes. If/when this bec
rsmith added inline comments.
Comment at: include/clang/AST/RecursiveASTVisitor.h:327
+
+ SmallVector getStmtChildren(CXXOperatorCallExpr *CE) {
+SmallVector Children(CE->children());
The copy here is more expensive than I'd like. Instead of returning a list
This revision was automatically updated to reflect the committed changes.
Closed by commit rL312965: [codeview] omit debug locations for nested exprs
unless column info enabled (authored by inglorion).
Changed prior to commit:
https://reviews.llvm.org/D37529?vs=114715&id=114716#toc
Repository:
Author: inglorion
Date: Mon Sep 11 15:11:57 2017
New Revision: 312965
URL: http://llvm.org/viewvc/llvm-project?rev=312965&view=rev
Log:
[codeview] omit debug locations for nested exprs unless column info enabled
Summary:
Microsoft Visual Studio expects debug locations to correspond to
statements.
inglorion updated this revision to Diff 114715.
inglorion added a comment.
renamed get{Nested,}ExpressionLocationsEnabled and moved it into
CodeGetModule.cpp
https://reviews.llvm.org/D37529
Files:
clang/lib/CodeGen/CGDebugInfo.cpp
clang/lib/CodeGen/CodeGenModule.cpp
clang/lib/CodeGen/Cod
rjmccall added a comment.
In https://reviews.llvm.org/D33826#866170, @lebedev.ri wrote:
> In https://reviews.llvm.org/D33826#866161, @JonasToth wrote:
>
> > There is an exception to the general rule (EXP36-C-EX2), stating that the
> > result of `malloc` and friends is allowed to be casted to str
Sorry, I completely missed that this affected the version string on
released binaries; I was even a reviewer on r312043 :-/ For Windows
release builds, the version info gets set explicitly, but not for
other builds.
We should have merged r312043 to 5.0.0.. perhaps it will go into 5.0.1.
- Hans
This revision was automatically updated to reflect the committed changes.
Closed by commit rL312958: [Driver] Support ubsan-minimal on Darwin (authored
by vedantk).
Changed prior to commit:
https://reviews.llvm.org/D37649?vs=114458&id=114704#toc
Repository:
rL LLVM
https://reviews.llvm.org/
This revision was automatically updated to reflect the committed changes.
Closed by commit rL312957: [ubsan-minimal] Document the new runtime (authored
by vedantk).
Changed prior to commit:
https://reviews.llvm.org/D37647?vs=114449&id=114703#toc
Repository:
rL LLVM
https://reviews.llvm.org/
Author: vedantk
Date: Mon Sep 11 14:37:06 2017
New Revision: 312958
URL: http://llvm.org/viewvc/llvm-project?rev=312958&view=rev
Log:
[Driver] Support ubsan-minimal on Darwin
Make it possible to use the minimal ubsan runtime on Darwin.
Differential Revision: https://reviews.llvm.org/D37649
Modi
Author: vedantk
Date: Mon Sep 11 14:37:05 2017
New Revision: 312957
URL: http://llvm.org/viewvc/llvm-project?rev=312957&view=rev
Log:
[ubsan-minimal] Document the new runtime
Differential Revision: https://reviews.llvm.org/D37647
Modified:
cfe/trunk/docs/ReleaseNotes.rst
cfe/trunk/docs/U
vsk closed this revision.
vsk added a comment.
Thanks, r312948
https://reviews.llvm.org/D37642
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
frutiger updated this revision to Diff 114701.
frutiger added a comment.
Use more specific target platform.
https://reviews.llvm.org/D37577
Files:
bindings/python/clang/cindex.py
bindings/python/tests/cindex/test_tls_kind.py
include/clang-c/Index.h
tools/libclang/CIndex.cpp
tools/libc
RKSimon added a comment.
In https://reviews.llvm.org/D37448#866700, @rnk wrote:
> lgtm
>
> I was hoping for a test case that didn't require assertions, but this is
> enough to land the fix.
My mistake, I should be able to remove that as well - I'll do it as part of the
commit. Thanks.
Repos
rnk added inline comments.
Comment at: clang/lib/CodeGen/CodeGenModule.h:32
#include "clang/Basic/XRayLists.h"
+#include "clang/Frontend/CodeGenOptions.h"
#include "llvm/ADT/DenseMap.h"
majnemer wrote:
> Any reason to do this? I'd just keep getNestedExpressionL
compnerd accepted this revision.
compnerd added inline comments.
This revision is now accepted and ready to land.
Comment at: bindings/python/tests/cindex/test_tls_kind.py:32
+__declspec(thread) int tls_declspec;
+""", lang = 'cpp', flags=['-fms-extensions', '-target', 'amd64-win
This revision was automatically updated to reflect the committed changes.
Closed by commit rL312947: [Lexer] Report more precise skipped regions
(PR34166) (authored by vedantk).
Changed prior to commit:
https://reviews.llvm.org/D36642?vs=114433&id=114676#toc
Repository:
rL LLVM
https://revi
Author: vedantk
Date: Mon Sep 11 13:47:45 2017
New Revision: 312948
URL: http://llvm.org/viewvc/llvm-project?rev=312948&view=rev
Log:
[pp-trace] Update skipped source ranges in tests
Depends on D36642
Modified:
clang-tools-extra/trunk/pp-trace/PPCallbacksTracker.cpp
clang-tools-extra/tru
Author: vedantk
Date: Mon Sep 11 13:47:42 2017
New Revision: 312947
URL: http://llvm.org/viewvc/llvm-project?rev=312947&view=rev
Log:
[Lexer] Report more precise skipped regions (PR34166)
This patch teaches the preprocessor to report more precise source ranges for
code that is skipped due to cond
majnemer added inline comments.
Comment at: clang/lib/CodeGen/CodeGenModule.h:32
#include "clang/Basic/XRayLists.h"
+#include "clang/Frontend/CodeGenOptions.h"
#include "llvm/ADT/DenseMap.h"
Any reason to do this? I'd just keep getNestedExpressionLocationsEnabl
This revision was automatically updated to reflect the committed changes.
Closed by commit rL312942: clang-rename: let -force handle multiple renames
(authored by vmiklos).
Changed prior to commit:
https://reviews.llvm.org/D37634?vs=114402&id=114663#toc
Repository:
rL LLVM
https://reviews.l
Author: vmiklos
Date: Mon Sep 11 13:18:38 2017
New Revision: 312942
URL: http://llvm.org/viewvc/llvm-project?rev=312942&view=rev
Log:
clang-rename: let -force handle multiple renames
Summary:
The use case is that renaming multiple symbols in a large enough codebase is
much faster if all of these
Author: compnerd
Date: Mon Sep 11 13:18:09 2017
New Revision: 312941
URL: http://llvm.org/viewvc/llvm-project?rev=312941&view=rev
Log:
Driver: default to `-fno-use-cxatexit` on Windows
This primarily impacts the Windows MSVC and Windows itanium
environments. Windows MSVC does not use `__cxa_atex
Nebiroth updated this revision to Diff 114654.
Nebiroth added a comment.
Simpilified a pointer return value.
https://reviews.llvm.org/D37150
Files:
clangd/ClangdLSPServer.cpp
clangd/ClangdLSPServer.h
clangd/ClangdServer.cpp
clangd/GlobalCompilationDatabase.cpp
clangd/GlobalCompilation
Nebiroth updated this revision to Diff 114652.
Nebiroth added a comment.
Return value when no file is found is now an empty string instead of file://
Minor code style changes and cleanup.
Ran clang-format on ClangdServer.h
https://reviews.llvm.org/D36150
Files:
clangd/ClangdLSPServer.cpp
cl
rnk added a comment.
In https://reviews.llvm.org/D37530#865333, @mstorsjo wrote:
> Attempted implementing Reid's suggestion.
>
> On the mingw-w64 mailing list
> (https://sourceforge.net/p/mingw-w64/mailman/message/36030072/), there were
> points made that it would be better with some mechanism
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
lgtm
https://reviews.llvm.org/D37530
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
lgtm
Comment at: clang/lib/CodeGen/CodeGenModule.h:517
+ /// Return true if we should emit location information for nested
expressions.
+ bool getNestedExpressionLocationsEnable
Nebiroth marked 5 inline comments as done.
Nebiroth added inline comments.
Comment at: clangd/ClangdLSPServer.cpp:234
+ else
+ResultUri = URI::fromFile("");
+
ilya-biryukov wrote:
> Running `unparse` on an instance created via `URI::fromFile("")` will resul
vivekvpandya updated this revision to Diff 114647.
vivekvpandya marked 8 inline comments as done.
https://reviews.llvm.org/D33514
Files:
include/llvm/Analysis/OptimizationDiagnosticInfo.h
include/llvm/CodeGen/MachineOptimizationRemarkEmitter.h
include/llvm/IR/DiagnosticHandler.h
include/l
vivekvpandya updated this revision to Diff 114645.
vivekvpandya added a comment.
Update
https://reviews.llvm.org/D37196
Files:
lib/CodeGen/CodeGenAction.cpp
Index: lib/CodeGen/CodeGenAction.cpp
===
--- lib/CodeGen/CodeGenAction
vivekvpandya marked an inline comment as done.
vivekvpandya added inline comments.
Comment at: lib/CodeGen/CodeGenAction.cpp:882-883
BEConsumer = Result.get();
-
+ VMContext->setDiagnosticHandler(llvm::make_unique(
+ CI.getCodeGenOpts(), Result.get()));
// Enable gene
yaxunl created this revision.
Herald added subscribers: t-tye, tpr, dstuttard, nhaehnle, wdng, kzhuravl.
https://reviews.llvm.org/D37703
Files:
lib/Basic/Targets/AMDGPU.h
test/CodeGenOpenCL/opencl_types.cl
Index: test/CodeGenOpenCL/opencl_types.cl
===
frutiger updated this revision to Diff 114640.
frutiger added a comment.
Add test cases for `__declspec(thread)` and static TLS. Clean up formatting to
adhere to the project style.
https://reviews.llvm.org/D37577
Files:
bindings/python/clang/cindex.py
bindings/python/tests/cindex/test_tls
Author: marshall
Date: Mon Sep 11 11:10:33 2017
New Revision: 312932
URL: http://llvm.org/viewvc/llvm-project?rev=312932&view=rev
Log:
mark mersenne_twister_engine<>::seed(result_type __sd) with
_LIBCPP_DISABLE_UBSAN_UNSIGNED_INTEGER_CHECK to placate UBSAN. Fixes PR#34160
Modified:
libcxx/
bcraig added a comment.
In https://reviews.llvm.org/D37677#866743, @Quuxplusone wrote:
> This current patch just swaps out std::mutex for a std::mutex-alike class
> that claims to be faster for uncontested accesses. Definitely safer than my
> interpretation. :) If this patch actually helps, the
rwols added a comment.
No, I don't have access to the repo, sorry forgot to mention this.
https://reviews.llvm.org/D37101
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
On Tue, Sep 5, 2017 at 2:47 PM Richard Smith via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> Author: rsmith
> Date: Tue Sep 5 14:46:22 2017
> New Revision: 312580
>
> URL: http://llvm.org/viewvc/llvm-project?rev=312580&view=rev
> Log:
> Fix memory leak after r312467. The ModuleMap is the o
Quuxplusone added a comment.
TODO.txt says "future should use for synchronization." I would have
interpreted this as meaning that the line
unsigned __state_;
should become
std::atomic __state_;
and appropriate loads and stores used so that `__is_ready()` can be checked
without having to
ilya-biryukov updated this revision to Diff 114634.
ilya-biryukov added a comment.
Fixed description of the change.
https://reviews.llvm.org/D37700
Files:
lib/Lex/PPDirectives.cpp
test/Index/preamble-conditionals-inverted-with-error.cpp
test/Index/preamble-conditionals-inverted.cpp
Inde
ilya-biryukov created this revision.
This fixes PR34547.
`Lexer::LexEndOfFile` handles recording of ConditionalStack for
preamble and reporting errors about unmatched conditionalal PP
directives.
However, SkipExcludedConditionalBlock contianed duplicated logic for
reporting errors and clearing Con
frutiger added inline comments.
Comment at: bindings/python/tests/cindex/test_tls_kind.py:14
+int tls_none;
+thread_local tls_dynamic;
+""", lang = 'cpp')
frutiger wrote:
> compnerd wrote:
> > Can we add a test case for static TLS as well please? Also, I think t
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
lgtm
I was hoping for a test case that didn't require assertions, but this is enough
to land the fix.
Repository:
rL LLVM
https://reviews.llvm.org/D37448
frutiger added inline comments.
Comment at: bindings/python/tests/cindex/test_tls_kind.py:14
+int tls_none;
+thread_local tls_dynamic;
+""", lang = 'cpp')
compnerd wrote:
> Can we add a test case for static TLS as well please? Also, I think that we
> should add
dennis.luxen added a comment.
In https://reviews.llvm.org/D37677#866362, @bcraig wrote:
> Is there a benchmark where this demonstrates some performance improvement? I
> fear that the switch to condition_variable_any will swamp any performance
> gains from the switch to a spin lock.
>
> Also, t
craig.topper added a comment.
Remove the builtins from BuiltinsX86.def
https://reviews.llvm.org/D37694
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Rakete abandoned this revision.
Rakete added a comment.
Agreed. Closing this.
https://reviews.llvm.org/D37667
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
craig.topper accepted this revision.
craig.topper added a comment.
This revision is now accepted and ready to land.
LGTM
https://reviews.llvm.org/D37562
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/li
craig.topper added a comment.
I think when you uploaded the changes to remove it from BuiltinsX86.def you
lost your earlier changes to the header files
https://reviews.llvm.org/D37668
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://l
compnerd requested changes to this revision.
compnerd added inline comments.
This revision now requires changes to proceed.
Comment at: lib/CMakeLists.txt:189
+macro(add_msvcrt_defs_if_needed target)
+ if(WIN32 AND NOT MINGW)
Please use `function` rather than
Author: marshall
Date: Mon Sep 11 09:05:42 2017
New Revision: 312923
URL: http://llvm.org/viewvc/llvm-project?rev=312923&view=rev
Log:
Add include of to , since things in there return
strings. Fixes PR#34529.
Added:
libcxx/trunk/test/std/diagnostics/syserr/syserr.errcode/syserr.errcode.ove
compnerd added inline comments.
Comment at: bindings/python/tests/cindex/test_tls_kind.py:14
+int tls_none;
+thread_local tls_dynamic;
+""", lang = 'cpp')
Can we add a test case for static TLS as well please? Also, I think that we
should add a test case for `__
spetrovic updated this revision to Diff 114618.
spetrovic added a comment.
Indentations fixed.
https://reviews.llvm.org/D34878
Files:
include/clang/Basic/DiagnosticDriverKinds.td
include/clang/Driver/CC1Options.td
include/clang/Driver/Options.td
lib/Driver/ToolChains/Arch/ARM.cpp
lib/
Typz added a comment.
ping?
https://reviews.llvm.org/D32478
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Typz added a comment.
ping?
https://reviews.llvm.org/D33440
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Typz added a comment.
ping?
https://reviews.llvm.org/D33589
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
krasimir added a comment.
Yes, I meant that it should at least be controlled by a flag. However, adding
flags to clang-format requires careful consideration, we usually would accept
options that are necessary for commonly used styles with style guides and with
a community willing to support the
krasimir updated this revision to Diff 114617.
krasimir added a comment.
- Add the original test cases
https://reviews.llvm.org/D37695
Files:
lib/Format/BreakableToken.cpp
lib/Format/BreakableToken.h
lib/Format/ContinuationIndenter.cpp
lib/Format/ContinuationIndenter.h
unittests/Forma
Author: mprobst
Date: Mon Sep 11 08:22:52 2017
New Revision: 312918
URL: http://llvm.org/viewvc/llvm-project?rev=312918&view=rev
Log:
clang-format: [JS] wrap and indent `goog.setTestOnly` calls.
Summary:
While `goog.setTestOnly` usually appears in the imports section of a file, it is
not actually
krasimir created this revision.
Herald added a subscriber: klimek.
This patch enables `BreakableToken` to manage the formatting of non-trailing
block comments. It is a refinement of https://reviews.llvm.org/D37007.
We discovered that the optimizer outsmarts us on cases where breaking the
comment
uriel.k created this revision.
This patch, together with a matching llvm patch
(https://reviews.llvm.org/D37693), implements the lowering of X86 ABS
intrinsics to IR.
https://reviews.llvm.org/D37694
Files:
lib/CodeGen/CGBuiltin.cpp
test/CodeGen/avx2-builtins.c
test/CodeGen/avx512bw-buil
erichkeane added a comment.
By my reading of the code, you still haven't fixed this example:
struct __declspec(uuid("---C000-0046")) IUnknown {};
struct IPropertyPageBase : public IUnknown {};
struct IPropertyPage : public IPropertyPageBase {};
__interface foo {
This revision was automatically updated to reflect the committed changes.
Closed by commit rL312917: [PCH] Allow VFS to be used for tests that generate
PCH files (authored by cameron314).
Changed prior to commit:
https://reviews.llvm.org/D37474?vs=114604&id=114612#toc
Repository:
rL LLVM
ht
Author: cameron314
Date: Mon Sep 11 08:03:23 2017
New Revision: 312917
URL: http://llvm.org/viewvc/llvm-project?rev=312917&view=rev
Log:
[PCH] Allow VFS to be used for tests that generate PCH files
When using a virtual file-system (VFS) and a preamble file (PCH) is generated,
it is generated on-d
cameron314 added inline comments.
Comment at: include/clang/Frontend/PrecompiledPreamble.h:100
/// PreambleBounds used to build the preamble
PreambleBounds getBounds() const;
ilya-biryukov wrote:
> Not introduced by this change, but could you also add a f
cameron314 updated this revision to Diff 114604.
cameron314 added a comment.
Final diff, will commit soon. Thanks!
https://reviews.llvm.org/D37474
Files:
include/clang/Frontend/PrecompiledPreamble.h
lib/Frontend/ASTUnit.cpp
unittests/Frontend/CMakeLists.txt
unittests/Frontend/PCHPreambl
chrib updated this revision to Diff 114589.
chrib added a comment.
Hello Eli,
You were right, using IsUnwindTablesDefault is the correct way to abstract the
need for UnwindTables. I missed the relationship between the driver
munwind-table and GodeGenOpts.UnwindTables use in Codegen.
Here is a
jina.nahias updated this revision to Diff 114594.
jina.nahias added a comment.
delete from include/clang/Basic/BuiltinsX86.def and
include/clang/Basic/BuiltinsX86_64.def
https://reviews.llvm.org/D37668
Files:
include/clang/Basic/BuiltinsX86.def
include/clang/Basic/BuiltinsX86_64.def
Inde
johannes added inline comments.
Comment at: include/clang/AST/RecursiveASTVisitor.h:334
+case OO_Arrow:
+case OO_Call:
+case OO_Subscript:
klimek wrote:
> Why do we need to swap for calls?
The idea is that the opening parenthesis/bracket comes after
alexfh accepted this revision.
alexfh added a comment.
Still LG. Fine to commit.
Thanks for the fix!
If you find time to construct the test case that triggers both affected code
paths (my test seems to only inspect one of them, IIUC), it would be nice to do
as a follow up.
https://reviews.ll
spetrovic added a comment.
Thanks for the review! I will check indentations with clang format.
https://reviews.llvm.org/D34878
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
On Fri, Sep 8, 2017 at 9:33 PM, Roman Lebedev via Phabricator via
cfe-commits wrote:
> lebedev.ri added a comment.
>
> In https://reviews.llvm.org/D37629#865068, @lebedev.ri wrote:
>
>> And finish reducing the code by for-range-loop`ing over array + use
>> `std::array`.
>
>
> I will need to fix h
kristof.beyls accepted this revision.
kristof.beyls added a comment.
This revision is now accepted and ready to land.
Thanks Strahinja!
I thought that some indentations looked a bit strange, so I'd just still check
that clang-format formats your changes the same way.
Otherwise LGTM!
https://rev
bcraig added a comment.
Is there a benchmark where this demonstrates some performance improvement? I
fear that the switch to condition_variable_any will swamp any performance gains
from the switch to a spin lock.
Also, the spin lock is being held during allocating operations (the exception
th
spetrovic updated this revision to Diff 114582.
https://reviews.llvm.org/D34878
Files:
include/clang/Basic/DiagnosticDriverKinds.td
include/clang/Driver/CC1Options.td
include/clang/Driver/Options.td
lib/Driver/ToolChains/Arch/ARM.cpp
lib/Driver/ToolChains/Arch/ARM.h
test/Driver/clang-
This revision was automatically updated to reflect the committed changes.
Closed by commit rL312911: [AST] Make RecursiveASTVisitor visit TemplateDecls
in source order (authored by krobelus).
Repository:
rL LLVM
https://reviews.llvm.org/D37662
Files:
cfe/trunk/include/clang/AST/LexicallyOrd
Author: lebedevri
Date: Mon Sep 11 06:12:31 2017
New Revision: 312912
URL: http://llvm.org/viewvc/llvm-project?rev=312912&view=rev
Log:
[clang-tidy] FunctionSizeCheck: wrap FunctionASTVisitor into anon namespace, NFC
This check is relatively simple, and is often being used
as an example. I'm awar
Author: krobelus
Date: Mon Sep 11 06:12:30 2017
New Revision: 312911
URL: http://llvm.org/viewvc/llvm-project?rev=312911&view=rev
Log:
[AST] Make RecursiveASTVisitor visit TemplateDecls in source order
Summary:
This causes template arguments to be traversed before the templated
declaration, which
ioeric added a comment.
Feel free to land the patch now if comments are addressed.
Repository:
rL LLVM
https://reviews.llvm.org/D36574
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-comm
arphaman added a comment.
@klimek I've added a patch that simplifies the interface and the template code
- https://reviews.llvm.org/D37681. Hopefully you'll find that the code there is
cleaner.
Repository:
rL LLVM
https://reviews.llvm.org/D36574
__
arphaman created this revision.
Herald added a subscriber: mgorny.
This patch tries to simplify the interface for the refactoring action rules and
refactoring requirements. It merges the selection constraints and selection
requirements into one class. The refactoring actions rules must now be
i
spetrovic added inline comments.
Comment at: lib/Driver/ToolChains/Arch/ARM.cpp:149-150
+else
+ D.Diag(diag::err_drv_invalid_mtp) << A->getAsString(Args);
+ }
+ return ReadTPMode::Soft;
kristof.beyls wrote:
> a return ReadTPMode::Invalid is missing her
kristof.beyls added inline comments.
Comment at: lib/Driver/ToolChains/Arch/ARM.cpp:137
+ const Driver &D = TC.getDriver();
+ arm::ReadTPMode ThreadPointer = ReadTPMode::Invalid;
+ if (Arg *A =
With the new version of the code, there's no need to have a Thread
spetrovic added a comment.
Comments addressed.
https://reviews.llvm.org/D34878
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
spetrovic updated this revision to Diff 114573.
https://reviews.llvm.org/D34878
Files:
include/clang/Basic/DiagnosticDriverKinds.td
include/clang/Driver/CC1Options.td
include/clang/Driver/Options.td
lib/Driver/ToolChains/Arch/ARM.cpp
lib/Driver/ToolChains/Arch/ARM.h
test/Driver/clang-
RKSimon updated this revision to Diff 114569.
RKSimon added a comment.
Added checks to test case
Repository:
rL LLVM
https://reviews.llvm.org/D37448
Files:
lib/CodeGen/CGStmt.cpp
test/CodeGen/pr34021.c
Index: test/CodeGen/pr34021.c
==
szepet updated this revision to Diff 114567.
szepet marked 3 inline comments as done.
szepet added a comment.
Updates based on the comments. Thanks for the reduced test case and the fast
review!
Not sure if in these cases am I allowed to commit (since it was accepted) or
since it was accepted a
1 - 100 of 127 matches
Mail list logo