Mind if we keep -fmodule-implementation-of as an alias so that we can gradually
transition to -fmodule-name ?
> On Feb 19, 2016, at 2:25 PM, Richard Smith via cfe-commits
> wrote:
>
> Author: rsmith
> Date: Fri Feb 19 16:25:36 2016
> New Revision: 261372
>
> URL: http://llvm.org/viewvc/llvm-p
weimingz updated this revision to Diff 49713.
weimingz added a comment.
Add "-f__FILE__-prefix-to-remove" flag to support the trim of the prefix.
Passing special value __ALL_DIR__ to remove all dir parts.
For example FILE is /a/b/c
-f__FILE__-prefix-to-remove=/a/ will cause FILE be expanded to b
Author: mzuckerm
Date: Thu Mar 3 02:55:20 2016
New Revision: 262593
URL: http://llvm.org/viewvc/llvm-project?rev=262593&view=rev
Log:
[Clang][AVX512][BUILTIN] Adding PSRL{W|WI}{128|256|512}
Differential Revision: http://reviews.llvm.org/D17754
Modified:
cfe/trunk/include/clang/Basic/Builti
Author: d0k
Date: Thu Mar 3 02:58:12 2016
New Revision: 262595
URL: http://llvm.org/viewvc/llvm-project?rev=262595&view=rev
Log:
[clang-tidy] Add "clang-tidy as a clang plugin" skeleton.
This doesn't really do much at the moment. You can load it via libclang
and set the -checks via an extra comm
Author: d0k
Date: Thu Mar 3 02:58:18 2016
New Revision: 262596
URL: http://llvm.org/viewvc/llvm-project?rev=262596&view=rev
Log:
[libclang] Link clang-tidy plugin into libclang if present.
This is a sad workaround for the lack of plugin support in libclang. Depends
on D17807, a tools-extra chang
This revision was automatically updated to reflect the committed changes.
Closed by commit rL262595: [clang-tidy] Add "clang-tidy as a clang plugin"
skeleton. (authored by d0k).
Changed prior to commit:
http://reviews.llvm.org/D17807?vs=49615&id=49718#toc
Repository:
rL LLVM
http://reviews.
This revision was automatically updated to reflect the committed changes.
Closed by commit rL262596: [libclang] Link clang-tidy plugin into libclang if
present. (authored by d0k).
Changed prior to commit:
http://reviews.llvm.org/D17808?vs=49616&id=49719#toc
Repository:
rL LLVM
http://review
nemanjai added inline comments.
Comment at: lib/Headers/altivec.h:136
@@ -131,3 +135,3 @@
#if defined(__POWER8_VECTOR__) && defined(__powerpc64__)
static vector double __ATTRS_o_ai vec_abs(vector double __a) {
I thought we were going to change the guard here t
Author: mzuckerm
Date: Thu Mar 3 03:26:01 2016
New Revision: 262598
URL: http://llvm.org/viewvc/llvm-project?rev=262598&view=rev
Log:
[CLANG][AVX512][BUILTIN] movdqa{32|64}{load|store|}{128|256|512}
Differential Revision: http://reviews.llvm.org/D17812
Modified:
cfe/trunk/include/clang/Bas
labath created this revision.
labath added a reviewer: akyrtzi.
labath added a subscriber: cfe-commits.
ASTImporter was importing this attribute only on destructor and constructor
decls, as they take
it in the constructor, but other decls can have this attribute as well, notably
the global
opera
labath added a subscriber: lldb-commits.
labath added a comment.
I noticed this while debugging an importing issue in LLDB.
I am not sure if I have selected the right reviewer, and also how/if to test
this...
http://reviews.llvm.org/D17845
___
cfe
Author: xazax
Date: Thu Mar 3 04:27:32 2016
New Revision: 262601
URL: http://llvm.org/viewvc/llvm-project?rev=262601&view=rev
Log:
[clang-tidy] Documentation fixes.
Modified:
clang-tools-extra/trunk/docs/clang-tidy/checks/misc-throw-by-value-catch-by-reference.rst
Modified:
clang-tools-ex
Author: alexfh
Date: Thu Mar 3 04:42:46 2016
New Revision: 262603
URL: http://llvm.org/viewvc/llvm-project?rev=262603&view=rev
Log:
[docs] Updated doxygen files to work well with doxygen 1.8.11
Doxygen 1.8.11 doesn't seem to like files with ".intro" extension by default.
Removed:
cfe/trunk/
hokein added a comment.
friendly ping.
Repository:
rL LLVM
http://reviews.llvm.org/D17762
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: alexfh
Date: Thu Mar 3 04:44:10 2016
New Revision: 262604
URL: http://llvm.org/viewvc/llvm-project?rev=262604&view=rev
Log:
[docs] Add missing file
Added:
cfe/trunk/docs/doxygen-mainpage.dox
Added: cfe/trunk/docs/doxygen-mainpage.dox
URL:
http://llvm.org/viewvc/llvm-project/cfe/tru
Author: alexfh
Date: Thu Mar 3 04:45:59 2016
New Revision: 262605
URL: http://llvm.org/viewvc/llvm-project?rev=262605&view=rev
Log:
[docs] Fix docs to work with doxygen 1.8.11
Added:
clang-tools-extra/trunk/docs/doxygen-mainpage.dox
Removed:
clang-tools-extra/trunk/docs/doxygen.intro
Mod
Author: chapuni
Date: Thu Mar 3 05:09:43 2016
New Revision: 262606
URL: http://llvm.org/viewvc/llvm-project?rev=262606&view=rev
Log:
Tweak CMakeLists not for libclang to depend on the variable
CLANG_TOOL_EXTRA_BUILD.
Modified:
cfe/trunk/tools/CMakeLists.txt
cfe/trunk/tools/libclang/CMak
bkramer added a comment.
This doesn't look right to me. Aborting tree transform on a bad source location
seems just wrong. Where is the invalid source location coming from? Also the
backtrace seems related to typo correction, maybe the bug is there?
Repository:
rL LLVM
http://reviews.llvm.o
sberg added a comment.
Can you please push this, I do not have commit access. Thanks
http://reviews.llvm.org/D16628
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sberg added a comment.
Can you please push this, I do not have commit access. Thanks
http://reviews.llvm.org/D15267
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
labath created this revision.
labath added a reviewer: nlewycky.
labath added subscribers: cfe-commits, lldb-commits.
While diagnosing a CXXNewExpr warning, we were calling isInSystemHeader(),
which expect to be
called with a valid source location. This causes an assertion failure if the
locatio
Author: mzuckerm
Date: Thu Mar 3 05:34:52 2016
New Revision: 262609
URL: http://llvm.org/viewvc/llvm-project?rev=262609&view=rev
Log:
[CLANG][AVX512][BUILTIN] movdqu{qi|hi} {128|256|512}
Differential Revision: http://reviews.llvm.org/D17814
Modified:
cfe/trunk/include/clang/Basic/Builtins
Author: marshall
Date: Thu Mar 3 06:04:39 2016
New Revision: 262610
URL: http://llvm.org/viewvc/llvm-project?rev=262610&view=rev
Log:
Fix for PR26812: possible overflow issue in std::allocator::allocate
Added:
libcxx/trunk/test/std/utilities/memory/default.allocator/allocator.members/alloca
Author: mzuckerm
Date: Thu Mar 3 06:17:50 2016
New Revision: 262611
URL: http://llvm.org/viewvc/llvm-project?rev=262611&view=rev
Log:
Fixing a checkfile error in avx512vlbw-builtins.c
Differential Revision: http://reviews.llvm.org/D17814
Modified:
cfe/trunk/test/CodeGen/avx512vlbw-builtins
richard.barton.arm added a subscriber: richard.barton.arm.
richard.barton.arm added a comment.
Hi Bradley
This matches the GCC targeting options:
https://sourceware.org/ml/binutils/2015-12/msg00295.html
https://sourceware.org/ml/binutils/2015-12/msg00296.html
The code changes LGTM. I think as th
a.sidorin updated this revision to Diff 49727.
a.sidorin marked 15 inline comments as done.
a.sidorin added a comment.
Add more comments; resolve issues pointed on review.
http://reviews.llvm.org/D17446
Files:
docs/LibASTMatchersReference.html
include/clang/ASTMatchers/ASTMatchers.h
inclu
Author: xazax
Date: Thu Mar 3 07:08:11 2016
New Revision: 262615
URL: http://llvm.org/viewvc/llvm-project?rev=262615&view=rev
Log:
[clang-tidy] Do not emit warnings from misc-suspicious-semicolon when the
compilation fails.
Added:
clang-tools-extra/trunk/test/clang-tidy/misc-suspicious-semi
xazax.hun added a comment.
The comments should be addressed in: http://reviews.llvm.org/rL262615
Repository:
rL LLVM
http://reviews.llvm.org/D16535
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/list
dsanders accepted this revision.
dsanders added a comment.
Thanks. LGTM
Repository:
rL LLVM
http://reviews.llvm.org/D16139
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
On 3 Mar 2016 3:18 a.m., "Argyrios Kyrtzidis via cfe-commits" <
cfe-commits@lists.llvm.org> wrote:
>
> Mind if we keep -fmodule-implementation-of as an alias so that we can
gradually transition to -fmodule-name ?
Sure, no problem.
> > On Feb 19, 2016, at 2:25 PM, Richard Smith via cfe-commits <
c
Author: stulova
Date: Thu Mar 3 07:33:19 2016
New Revision: 262616
URL: http://llvm.org/viewvc/llvm-project?rev=262616&view=rev
Log:
[OpenCL] Apply missing restrictions for Blocks in OpenCL v2.0
Applying the following restrictions for block types in OpenCL (v2.0 s6.12.5):
- __block storage clas
On Thu, Mar 3, 2016 at 2:08 PM, Gabor Horvath via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> Author: xazax
> Date: Thu Mar 3 07:08:11 2016
> New Revision: 262615
>
> URL: http://llvm.org/viewvc/llvm-project?rev=262615&view=rev
> Log:
> [clang-tidy] Do not emit warnings from misc-suspiciou
rob.lougher added a comment.
Ping. Please can somebody review this? Thanks!
http://reviews.llvm.org/D16298
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: mzuckerm
Date: Thu Mar 3 07:43:05 2016
New Revision: 262617
URL: http://llvm.org/viewvc/llvm-project?rev=262617&view=rev
Log:
[CLANG][AVX512][BUILTIN] movddup{128|256|512}
Differential Revision: http://reviews.llvm.org/D17826
Modified:
cfe/trunk/include/clang/Basic/BuiltinsX86.def
Author: xazax
Date: Thu Mar 3 07:43:23 2016
New Revision: 262618
URL: http://llvm.org/viewvc/llvm-project?rev=262618&view=rev
Log:
[clang-tidy] Improve the robustness of a test.
Modified:
clang-tools-extra/trunk/test/clang-tidy/misc-suspicious-semicolon-fail.cpp
Modified:
clang-tools-extra
Thank you for the suggestions. This should be addressed in r262618.
On 3 March 2016 at 14:42, Alexander Kornienko wrote:
>
>
> On Thu, Mar 3, 2016 at 2:08 PM, Gabor Horvath via cfe-commits <
> cfe-commits@lists.llvm.org> wrote:
>
>> Author: xazax
>> Date: Thu Mar 3 07:08:11 2016
>> New Revision
bsmith closed this revision.
bsmith added a comment.
Committed as r262619.
Repository:
rL LLVM
http://reviews.llvm.org/D15283
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: brasmi01
Date: Thu Mar 3 07:52:22 2016
New Revision: 262619
URL: http://llvm.org/viewvc/llvm-project?rev=262619&view=rev
Log:
[ARM] Add Clang targeting for ARMv8-M Baseline/Mainline
Modified:
cfe/trunk/lib/Basic/Targets.cpp
cfe/trunk/lib/Driver/Tools.cpp
cfe/trunk/test/Driver
kparzysz added a comment.
This seemed fine to me, but at the second thought, the lines 2610-2613 in the
original should be removed.
http://reviews.llvm.org/D17784
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin
amehsan added inline comments.
Comment at: lib/Headers/altivec.h:136
@@ -131,3 +135,3 @@
#if defined(__POWER8_VECTOR__) && defined(__powerpc64__)
static vector double __ATTRS_o_ai vec_abs(vector double __a) {
nemanjai wrote:
> I thought we were going to change
hfinkel added a subscriber: hfinkel.
hfinkel added a comment.
In http://reviews.llvm.org/D17741#367113, @weimingz wrote:
> Add "-f__FILE__-prefix-to-remove" flag to support the trim of the prefix.
> Passing special value __ALL_DIR__ to remove all dir parts.
>
> For example FILE is /a/b/c
> -f_
ioeric abandoned this revision.
ioeric added a comment.
Start a new revision to put applyAllReplacementsAndFormat into non-core
libTooling .
http://reviews.llvm.org/D17761
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.or
I spoke with a Microsoft compiler engineer who said this is most
likely a bug that static_assert works in C mode in MSVC. Based on
that, I would strongly prefer to not accept this patch.
~Aaron
On Wed, Mar 2, 2016 at 9:33 PM, Aaron Ballman wrote:
> I will talk to someone about this tomorrow to s
ioeric created this revision.
ioeric added reviewers: klimek, djasper.
ioeric added a subscriber: cfe-commits.
Herald added a subscriber: klimek.
formatAndApplyAllReplacements takes a set of Replacements, applies them on a
Rewriter, and reformats the changed code.
http://reviews.llvm.org/D17852
Now I'm completely confused... :-)
Can we rely that this MS engineer has enough authority to declare this
to be a bug?
And more importantly -- is MS willing to fix the [supposed] bug in
future MSVC compilers? I frankly don't think so...
Either way, it's up to Reid to decide what to do. Reid?
Yo
On Thu, Mar 3, 2016 at 10:43 AM, Andrey Bokhanko
wrote:
> Now I'm completely confused... :-)
>
> Can we rely that this MS engineer has enough authority to declare this
> to be a bug?
He's on the compiler team, so yes.
> And more importantly -- is MS willing to fix the [supposed] bug in
> future
Author: sfantao
Date: Thu Mar 3 10:20:23 2016
New Revision: 262625
URL: http://llvm.org/viewvc/llvm-project?rev=262625&view=rev
Log:
[OpenMP] Code generation for teams - kernel launching
Summary:
This patch implements the launching of a target region in the presence of a
nested teams region, i.
On Thu, Mar 3, 2016 at 7:52 AM, Aaron Ballman
wrote:
> On Thu, Mar 3, 2016 at 10:43 AM, Andrey Bokhanko
> wrote:
> > Now I'm completely confused... :-)
> >
> > Can we rely that this MS engineer has enough authority to declare this
> > to be a bug?
>
> He's on the compiler team, so yes.
>
> > And
nemanjai added a comment.
All of my comments are just nits and shouldn't hold up approval. As far as I
can tell this looks fine, but I'll let the LGTM come from Kit or Hal.
Comment at: lib/Headers/altivec.h:136
@@ -131,3 +135,3 @@
#if defined(__POWER8_VECTOR__) && defined(__
alexfh accepted this revision.
alexfh added a comment.
This revision is now accepted and ready to land.
LG
Repository:
rL LLVM
http://reviews.llvm.org/D17849
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/ma
Thank you!
On Thu, Mar 3, 2016 at 2:48 PM, Gábor Horváth wrote:
> Thank you for the suggestions. This should be addressed in r262618.
>
> On 3 March 2016 at 14:42, Alexander Kornienko wrote:
>
>>
>>
>> On Thu, Mar 3, 2016 at 2:08 PM, Gabor Horvath via cfe-commits <
>> cfe-commits@lists.llvm.org
aaboud created this revision.
aaboud added reviewers: hjl.tools, DavidKreitzer, ABataev.
aaboud added a subscriber: cfe-commits.
Resolved Bug 26414.
https://llvm.org/bugs/show_bug.cgi?id=26414
Since interrupt handler must be returned with iretq, tail call can't be used.
http://reviews.llvm.org/
Author: djasper
Date: Thu Mar 3 11:34:14 2016
New Revision: 262630
URL: http://llvm.org/viewvc/llvm-project?rev=262630&view=rev
Log:
clang-format: Use stable_sort when sorting #includes.
Otherwise, clang-format can output useless replacements in the presence
of identical #includes
Modified:
sfantao updated this revision to Diff 49749.
sfantao marked 4 inline comments as done.
sfantao added a comment.
- Remove extra braces and revert the sense of the whole/unity array section
check.
http://reviews.llvm.org/D17547
Files:
include/clang/Basic/DiagnosticSemaKinds.td
lib/Sema/SemaO
sfantao added inline comments.
Comment at: lib/Sema/SemaOpenMP.cpp:9071
@@ +9070,3 @@
+/// 0 if it is inconclusive.
+static int CheckArrayExpressionReferToWholeSize(Sema &SemaRef, const Expr *E,
+QualType BaseQTy) {
djasper added inline comments.
Comment at: include/clang/Tooling/Core/Replacement.h:237
@@ +236,3 @@
+/// related to the same file entry are put into the same vector.
+FileToReplacementsMap groupReplacementsByFile(const Replacements &Replaces,
+
On Thu, Mar 3, 2016 at 12:06 PM, David Majnemer
wrote:
>
>
> On Thu, Mar 3, 2016 at 7:52 AM, Aaron Ballman
> wrote:
>>
>> On Thu, Mar 3, 2016 at 10:43 AM, Andrey Bokhanko
>> wrote:
>> > Now I'm completely confused... :-)
>> >
>> > Can we rely that this MS engineer has enough authority to declare
Author: stulova
Date: Thu Mar 3 12:38:40 2016
New Revision: 262641
URL: http://llvm.org/viewvc/llvm-project?rev=262641&view=rev
Log:
[OpenCL] Improve diagnostics of address spaces for variables in function
- Prevent local variables to be declared in global AS
- Diagnose AS of local variables w
sbenza added a comment.
In http://reviews.llvm.org/D17811#366482, @Eugene.Zelenko wrote:
> Does it make http://reviews.llvm.org/D17772 obsolete?
Yes. The other patch has already been abandoned.
http://reviews.llvm.org/D17811
___
cfe-commits maili
weimingz updated the summary for this revision.
weimingz updated this revision to Diff 49762.
weimingz added a comment.
Change the option name to -ffile-macro-prefix-to-remove
http://reviews.llvm.org/D17741
Files:
include/clang/Driver/Options.td
include/clang/Lex/Preprocessor.h
include/cl
manmanren created this revision.
manmanren added reviewers: dexonsmith, rjmccall.
manmanren added a subscriber: cfe-commits.
This can be used to display Fix-Its.
We only add this to GNU attributes. Since it now has two optional arguments,
and the common parsing does not handle "replacement = ", w
Author: cbertol
Date: Thu Mar 3 14:34:23 2016
New Revision: 262652
URL: http://llvm.org/viewvc/llvm-project?rev=262652&view=rev
Log:
Add code generation for teams directive inside target region
Modified:
cfe/trunk/lib/CodeGen/CGOpenMPRuntime.cpp
cfe/trunk/lib/CodeGen/CGOpenMPRuntime.h
carlo.bertolli closed this revision.
carlo.bertolli added a comment.
Committed revision 262652.
Repository:
rL LLVM
http://reviews.llvm.org/D17148
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listi
Nico,
the tests for ASTImporter (currently, test/ASTMerge) are currently set up to do
an AST merge and check that certain errors occur. I don’t believe they
actually go and code-gen, which would have caught this.
I think investing in better ASTImporter testing is a great goal, but right now
L
tstellarAMD added a comment.
Do you have a test case for this?
http://reviews.llvm.org/D17552
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: sylvestre
Date: Thu Mar 3 14:57:16 2016
New Revision: 262655
URL: http://llvm.org/viewvc/llvm-project?rev=262655&view=rev
Log:
fix some minor typos in the doc
Modified:
clang-tools-extra/trunk/docs/clang-tidy/checks/misc-string-integer-assignment.rst
clang-tools-extra/trunk/doc
Author: sylvestre
Date: Thu Mar 3 14:54:26 2016
New Revision: 262654
URL: http://llvm.org/viewvc/llvm-project?rev=262654&view=rev
Log:
Fix two minor syntax issues in the documentation
Modified:
clang-tools-extra/trunk/docs/clang-tidy/checks/misc-forward-declaration-namespace.rst
clang-
Author: dcoughlin
Date: Thu Mar 3 15:38:39 2016
New Revision: 262659
URL: http://llvm.org/viewvc/llvm-project?rev=262659&view=rev
Log:
[analyzer] ObjCDeallocChecker: Only check for nil-out when type is retainable.
This fixes a crash when setting a property of struct type in -dealloc.
Modified:
hintonda updated this revision to Diff 49772.
hintonda added a comment.
Improve efficiency by only keeping track of seen fields, and only when
designators have been used.
http://reviews.llvm.org/D17407
Files:
lib/Sema/SemaInit.cpp
test/SemaCXX/cxx0x-initializer-constructor.cpp
Index: test/
On Thu, Mar 03, 2016 at 07:39:04PM +, Weiming Zhao via cfe-commits wrote:
> Change the option name to -ffile-macro-prefix-to-remove
This still sounds to me like a solution for a very restricted part of a
much more generic problem...
Joerg
___
cfe-co
On Thu, Mar 3, 2016 at 1:50 PM, Joerg Sonnenberger via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> On Thu, Mar 03, 2016 at 07:39:04PM +, Weiming Zhao via cfe-commits
> wrote:
> > Change the option name to -ffile-macro-prefix-to-remove
>
> This still sounds to me like a solution for a ve
kparzysz added a comment.
This seemed fine to me, but at the second thought, the lines 2610-2613 in the
original should be removed.
http://reviews.llvm.org/D17784
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin
Author: cbertol
Date: Thu Mar 3 16:09:40 2016
New Revision: 262663
URL: http://llvm.org/viewvc/llvm-project?rev=262663&view=rev
Log:
[OPENMP] firstprivate and private clauses of teams, host codegeneration
Add code generation support for firstprivate and private clauses of teams on
the host. Add
carlo.bertolli closed this revision.
carlo.bertolli added a comment.
Committed revision 262663.
Repository:
rL LLVM
http://reviews.llvm.org/D17582
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listi
chandlerc added a comment.
In http://reviews.llvm.org/D17784#367601, @kparzysz wrote:
> This seemed fine to me, but at the second thought, the lines 2610-2613 in the
> original should be removed.
I agree. I think using LLVM_PREFIX here in any capacity is incorrect.
http://reviews.llvm.org/D1
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
LGTM
Comment at: lib/Sema/SemaOverload.cpp:1966-1967
@@ -1955,4 +1965,4 @@
// C99 6.3.1.5p1:
// When a float is promoted to double or long double, or a
As a first patch I think you can just leave every
UserLabelPrefix = ""
in place.
Cheers,
Rafael
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
yaxunl updated the summary for this revision.
yaxunl added a reviewer: tstellarAMD.
yaxunl removed a subscriber: tstellarAMD.
yaxunl updated this revision to Diff 49781.
yaxunl added a comment.
Add a test for -backend-option with and w/o target machine as suggested by Tom.
http://reviews.llvm.or
mclow.lists accepted this revision.
mclow.lists added a comment.
This revision is now accepted and ready to land.
Yes, sorry.
http://reviews.llvm.org/D15539
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailma
Sorry for not following up here; this will actually be solved in an
entirely different way by http://reviews.llvm.org/D17183
On Thu, Mar 3, 2016 at 5:53 PM, Rafael Espíndola
wrote:
> As a first patch I think you can just leave every
>
> UserLabelPrefix = ""
>
> in place.
>
> Cheers,
> Rafael
___
rafael added a subscriber: rafael.
rafael added a comment.
This is awesome!
Comment at: include/clang/CodeGen/BackendUtil.h:38
@@ -37,3 +37,3 @@
const TargetOptions &TOpts, const LangOptions &LOpts,
- StringRef TDesc, llvm::Modul
On Thu, Mar 03, 2016 at 02:09:17PM -0800, Nico Weber via cfe-commits wrote:
> On Thu, Mar 3, 2016 at 1:50 PM, Joerg Sonnenberger via cfe-commits <
> cfe-commits@lists.llvm.org> wrote:
>
> > On Thu, Mar 03, 2016 at 07:39:04PM +, Weiming Zhao via cfe-commits
> > wrote:
> > > Change the option na
On Thu, Mar 3, 2016 at 4:28 PM, Joerg Sonnenberger via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> On Thu, Mar 03, 2016 at 02:09:17PM -0800, Nico Weber via cfe-commits wrote:
> > On Thu, Mar 3, 2016 at 1:50 PM, Joerg Sonnenberger via cfe-commits <
> > cfe-commits@lists.llvm.org> wrote:
> >
arsenm added inline comments.
Comment at: lib/CodeGen/CGCall.cpp:1601
@@ +1600,3 @@
+if (CodeGenOpts.AMDGPUToolsInsertNopsOpt)
+ FuncAttrs.addAttribute("amdgpu_tools_insert_nops");
+if (CodeGenOpts.AMDGPUToolsNumReservedVGPROpt)
Yes
http://reviews.l
ryee88 added a comment.
Thanks! I don’t have commit permissions; can you submit for me?
http://reviews.llvm.org/D16949
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
srhines created this revision.
srhines added reviewers: cfe-commits, weimingz.
Herald added a subscriber: aemerson.
Using -no-integrated-as causes -mcpu=krait to be transformed into
-march=armv7-a today. This precludes the assembler from using
instructions like sdiv, which are present for krait. C
arpith-jacob created this revision.
arpith-jacob added reviewers: ABataev, hfinkel.
arpith-jacob added subscribers: cfe-commits, carlo.bertolli, sfantao, kkwli0,
caomhin, fraggamuffin.
Herald added a subscriber: jholewinski.
This patch adds base support for codegen of the target directive on the
Author: akirtzidis
Date: Thu Mar 3 22:24:32 2016
New Revision: 262686
URL: http://llvm.org/viewvc/llvm-project?rev=262686&view=rev
Log:
[index] Ignore ObjCTypeParamDecls during indexing.
Modified:
cfe/trunk/lib/Index/IndexingContext.cpp
cfe/trunk/test/Index/Core/index-source.m
Modified:
ABataev added inline comments.
Comment at: lib/Sema/SemaOpenMP.cpp:9127
@@ +9126,3 @@
+// whose base type is \a BaseQTy.
+static int CheckArrayExpressionDoesNotReferToUnitySize(Sema &SemaRef,
+ const Expr *E,
T
ABataev accepted this revision.
ABataev added a comment.
This revision is now accepted and ready to land.
LG
http://reviews.llvm.org/D17853
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-co
weimingz added a reviewer: apazos.
weimingz added a comment.
LGTM. Ana, could you take a look too?
http://reviews.llvm.org/D17874
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ABataev added inline comments.
Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:3790
@@ -3923,3 +3789,3 @@
// where DD_ is an ID unique to the file (device and file IDs), PP is the
- // mangled name of the function that encloses the target region and BB is the
+ // mangled nam
Author: majnemer
Date: Thu Mar 3 23:26:14 2016
New Revision: 262687
URL: http://llvm.org/viewvc/llvm-project?rev=262687&view=rev
Log:
[VFS] Switch from close to SafelyCloseFileDescriptor
The SafelyCloseFileDescriptor machinery does the right thing in the face
of signals while close will do somet
Author: majnemer
Date: Thu Mar 3 23:26:16 2016
New Revision: 262688
URL: http://llvm.org/viewvc/llvm-project?rev=262688&view=rev
Log:
[X86] Pass __m64 types via SSE registers for GCC compatibility
For compatibility with GCC, classify __m64 as SSE.
However, clang is a platform compiler for certai
pxli168 updated this revision to Diff 49797.
http://reviews.llvm.org/D16876
Files:
include/clang/Basic/Builtins.h
include/clang/Basic/DiagnosticSemaKinds.td
lib/Sema/SemaChecking.cpp
test/SemaOpenCL/invalid-pipe-builtin-cl2.0.cl
Index: test/SemaOpenCL/invalid-pipe-builtin-cl2.0.cl
==
> To: cfe-commits@lists.llvm.org
> Subject: r262688 - [X86] Pass __m64 types via SSE registers for GCC
> compatibility
>
> Author: majnemer
> Date: Thu Mar 3 23:26:16 2016
> New Revision: 262688
>
> URL: http://llvm.org/viewvc/llvm-project?rev=262688&view=rev
> Log:
> [X86] Pass __m64 types via
Author: pxl
Date: Fri Mar 4 01:11:16 2016
New Revision: 262692
URL: http://llvm.org/viewvc/llvm-project?rev=262692&view=rev
Log:
[OpenCL] Refine pipe builtin support
Summary:
Refine the type builtin support as the request with
http://lists.llvm.org/pipermail/cfe-commits/Week-of-Mon-20160201/1486
Author: akirtzidis
Date: Fri Mar 4 01:17:48 2016
New Revision: 262694
URL: http://llvm.org/viewvc/llvm-project?rev=262694&view=rev
Log:
[index] Include parameter types in the USRs for C functions marked with
'overloadable' attribute.
Modified:
cfe/trunk/lib/Index/USRGeneration.cpp
cfe/t
Author: akirtzidis
Date: Fri Mar 4 01:17:43 2016
New Revision: 262693
URL: http://llvm.org/viewvc/llvm-project?rev=262693&view=rev
Log:
[index] In ObjC++ handle objc type parameters for function USRs.
Added:
cfe/trunk/test/Index/Core/index-source.mm
Modified:
cfe/trunk/lib/Index/USRGener
Author: akirtzidis
Date: Fri Mar 4 01:17:53 2016
New Revision: 262695
URL: http://llvm.org/viewvc/llvm-project?rev=262695&view=rev
Log:
[index] Distinguish USRs of anonymous enums by using their first enumerator.
rdar://24609949.
Modified:
cfe/trunk/lib/Index/IndexingContext.cpp
cfe/tru
1 - 100 of 101 matches
Mail list logo
