george.burgess.iv created this revision.
As it turns out, emitting diagnostics from places where you're not meant to
emit them from is a very bad idea. :)
After some looking around, it seems that it's less insane to check for
`diagnose_if` attributes in code that's already checking for e.g. nul
hintonda added a comment.
Thanks for all the feedback. I've tried to address all your comments, and
reworked the documentation. Please let me know if I missed or misunderstood
anything.
https://reviews.llvm.org/D20693
___
cfe-commits mailing lis
hintonda updated this revision to Diff 84926.
hintonda marked 22 inline comments as done.
hintonda added a comment.
- Add support for matching MemberPointerType's.
- Add noexcept(false) option plus test; allow invalid replacement ranges;
enhance diagnostics.
- Updated docs.
https://reviews.llvm
On Jan 18, 2017 7:34 PM, "Akira Hatanaka via Phabricator" <
revi...@reviews.llvm.org> wrote:
ahatanak added a comment.
In https://reviews.llvm.org/D28467#649861, @krasin wrote:
> This change makes Clang hardly incompatible with MSVC++. Consider the
following program:
>
> #include
>
> int ma
compnerd added a comment.
I wonder if we should take this a bit further. Use something like a
`PAGE_SIZE` constant to avoid this confusion entirely. Furthremore, different
targets could use different page sizes. IIRC, SPARC has a 8K page size by
default.
Repository:
rL LLVM
https://revi
compnerd added a comment.
While I love this direction (the original version really was an unintelligible
pile of code), I really think that this change may be taking on too much. Why
not split it up first and do nothing else. We could do the MS ABI
implementation in a subsequent change. This
compnerd added a comment.
I think that the checks should be exact. Instead, we should check the
compilation mode and change the exact value (i.e. if we are using i387 mode, we
would use the adjusted value).
Repository:
rL LLVM
https://reviews.llvm.org/D28862
Merged in r292463.
Cheers,
Hans
On Tue, Jan 17, 2017 at 9:25 PM, Richard Smith wrote:
> Yes, let's.
>
> On 17 Jan 2017 9:11 pm, "Hans Wennborg" wrote:
>
> What do you think; time to merge it?
>
> On Fri, Jan 13, 2017 at 1:21 PM, Richard Smith
> wrote:
>> I'd like to wait a bit to see how the d
Author: dehao
Date: Wed Jan 18 18:44:21 2017
New Revision: 292458
URL: http://llvm.org/viewvc/llvm-project?rev=292458&view=rev
Log:
Add -fdebug-info-for-profiling to emit more debug info for sample pgo profile
collection
Summary:
SamplePGO uses profile with debug info to collect profile. Unlike
ahatanak added a comment.
In https://reviews.llvm.org/D28467#649861, @krasin wrote:
> This change makes Clang hardly incompatible with MSVC++. Consider the
> following program:
>
> #include
>
> int main(void) {
> const int kDelta = 1001;
> auto g = [kDelta](int i)
>
smeenai added a comment.
@EricWF and I discussed this on IRC a bunch yesterday.
We were inadvertently statically linking the Microsoft C++ libraries into
tests, which is why `std::set_new_handler` was working with this patch. With
that incorrect link removed, we get an undefined reference to
`
echristo accepted this revision.
echristo added a comment.
Thanks for explaining all of this and going through it Dehao.
LGTM.
-eric
https://reviews.llvm.org/D25435
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-
pcc updated this revision to Diff 84907.
pcc added a comment.
Herald added a subscriber: mgorny.
- Add missing test dependency
https://reviews.llvm.org/D28843
Files:
clang/lib/CodeGen/BackendUtil.cpp
clang/test/CMakeLists.txt
clang/test/CodeGenCXX/type-metadata-thinlto.cpp
Index: clang/
Author: pcc
Date: Wed Jan 18 18:04:44 2017
New Revision: 292450
URL: http://llvm.org/viewvc/llvm-project?rev=292450&view=rev
Log:
Add llvm-dis dependency to check-clang.
Modified:
cfe/trunk/test/CMakeLists.txt
Modified: cfe/trunk/test/CMakeLists.txt
URL:
http://llvm.org/viewvc/llvm-project/
tejohnson accepted this revision.
tejohnson added a comment.
This revision is now accepted and ready to land.
LGTM
https://reviews.llvm.org/D28843
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo
Author: pcc
Date: Wed Jan 18 17:55:27 2017
New Revision: 292448
URL: http://llvm.org/viewvc/llvm-project?rev=292448&view=rev
Log:
Move vtable type metadata emission behind a cc1-level flag.
In ThinLTO mode, type metadata will require the module to be written as a
multi-module bitcode file, which
pcc updated this revision to Diff 84903.
pcc added a comment.
Refresh
https://reviews.llvm.org/D28843
Files:
clang/lib/CodeGen/BackendUtil.cpp
clang/test/CodeGenCXX/type-metadata-thinlto.cpp
Index: clang/test/CodeGenCXX/type-metadata-thinlto.cpp
===
This revision was automatically updated to reflect the committed changes.
Closed by commit rL292448: Move vtable type metadata emission behind a
cc1-level flag. (authored by pcc).
Changed prior to commit:
https://reviews.llvm.org/D28877?vs=84900&id=84902#toc
Repository:
rL LLVM
https://revi
mehdi_amini accepted this revision.
mehdi_amini added a comment.
This revision is now accepted and ready to land.
LGTM.
https://reviews.llvm.org/D28877
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/lis
pcc added inline comments.
Comment at: clang/test/CodeGenCXX/type-metadata-thinlto.cpp:2
+// RUN: %clang_cc1 -flto=thin -triple x86_64-unknown-linux -fvisibility hidden
-emit-llvm-bc -o %t %s
+// RUN: llvm-modextract -o - -n 1 %t | llvm-dis | FileCheck %s
+
tejo
pcc created this revision.
In ThinLTO mode, type metadata will require the module to be written as a
multi-module bitcode file, which is currently incompatible with the Darwin
linker. It is also useful to be able to enable or disable multi-module bitcode
for testing purposes. This introduces a cc1
malcolm.parsons added a comment.
MSVC's behaviour is discussed here:
https://social.msdn.microsoft.com/Forums/SqlServer/en-US/4abf18bd-4ae4-4c72-ba3e-3b13e7909d5f/error-c2057-or-c3493-trying-to-use-an-integral-constant-expression-inside-a-lambda?forum=vclanguage
and here:
https://connect.microsof
Author: stl_msft
Date: Wed Jan 18 16:19:14 2017
New Revision: 292443
URL: http://llvm.org/viewvc/llvm-project?rev=292443&view=rev
Log:
[libcxx] [test] Add msvc_stdlib_force_include.hpp.
No functional change; nothing includes this, instead our test harness
injects it via the /FI compiler option.
mehdi_amini added a comment.
In https://reviews.llvm.org/D28867#649760, @vsk wrote:
> Ah ok, so it sounds like a better approach would be to split the missing
> record message into a separate off-by-default warning.
I'm in favor of this.
https://reviews.llvm.org/D28867
___
krasin added a comment.
This change makes Clang hardly incompatible with MSVC++. Consider the following
program:
#include
int main(void) {
const int kDelta = 1001;
auto g = [kDelta](int i)
{
printf("%d\n", i % kDelta);
};
g(2);
}
tejohnson added inline comments.
Comment at: clang/test/CodeGenCXX/type-metadata-thinlto.cpp:2
+// RUN: %clang_cc1 -flto=thin -triple x86_64-unknown-linux -fvisibility hidden
-emit-llvm-bc -o %t %s
+// RUN: llvm-modextract -o - -n 1 %t | llvm-dis | FileCheck %s
+
---
manmanren added a comment.
Ping :]
Hoping to wrap this up this week.
Cheers,
Manman
https://reviews.llvm.org/D28299
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
manmanren marked an inline comment as done.
manmanren added a comment.
Ping :]
I am hoping to wrap this up this week. Thanks,
Manman
https://reviews.llvm.org/D27689
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bi
mehdi_amini added inline comments.
Comment at: clang/test/CodeGenCXX/type-metadata-thinlto.cpp:2
+// RUN: %clang_cc1 -flto=thin -triple x86_64-unknown-linux -fvisibility hidden
-emit-llvm-bc -o %t %s
+// RUN: llvm-modextract -o - -n 1 %t | llvm-dis | FileCheck %s
+
-
tejohnson added inline comments.
Comment at: clang/test/CodeGenCXX/type-metadata-thinlto.cpp:2
+// RUN: %clang_cc1 -flto=thin -triple x86_64-unknown-linux -fvisibility hidden
-emit-llvm-bc -o %t %s
+// RUN: llvm-modextract -o - -n 1 %t | llvm-dis | FileCheck %s
+
---
mehdi_amini added inline comments.
Comment at: clang/test/CodeGenCXX/type-metadata-thinlto.cpp:2
+// RUN: %clang_cc1 -flto=thin -triple x86_64-unknown-linux -fvisibility hidden
-emit-llvm-bc -o %t %s
+// RUN: llvm-modextract -o - -n 1 %t | llvm-dis | FileCheck %s
+
-
pcc added inline comments.
Comment at: clang/test/CodeGenCXX/type-metadata-thinlto.cpp:2
+// RUN: %clang_cc1 -flto=thin -triple x86_64-unknown-linux -fvisibility hidden
-emit-llvm-bc -o %t %s
+// RUN: llvm-modextract -o - -n 1 %t | llvm-dis | FileCheck %s
+
mehd
Author: dblaikie
Date: Wed Jan 18 15:15:18 2017
New Revision: 292439
URL: http://llvm.org/viewvc/llvm-project?rev=292439&view=rev
Log:
Remove now redundant code that ensured debug info for class definitions was
emitted under certain circumstances
Introduced in r181561 - it may've been subsumed b
mehdi_amini added inline comments.
Comment at: clang/test/CodeGenCXX/type-metadata-thinlto.cpp:2
+// RUN: %clang_cc1 -flto=thin -triple x86_64-unknown-linux -fvisibility hidden
-emit-llvm-bc -o %t %s
+// RUN: llvm-modextract -o - -n 1 %t | llvm-dis | FileCheck %s
+
-
jroelofs added a comment.
In https://reviews.llvm.org/D28820#649770, @efriedma wrote:
> > There would be a big performance penalty for ISRs with callees that don't
> > use VFP regs.
>
> Sacrificing correctness for the sake of performance seems like a bad idea...
I don't quite see it that way,
pcc added inline comments.
Comment at: clang/lib/CodeGen/BackendUtil.cpp:694
+else
+ PerModulePasses.add(
+ createBitcodeWriterPass(*OS, CodeGenOpts.EmitLLVMUseLists));
tejohnson wrote:
> Can we transform other callers of createBitcodeWriterPass
This revision was automatically updated to reflect the committed changes.
Closed by commit rL292437: [OpenMP] Support for the if-clause on the combined
directive 'target parallel'. (authored by arpith).
Changed prior to commit:
https://reviews.llvm.org/D28781?vs=84816&id=84883#toc
Repository:
Author: graydon
Date: Wed Jan 18 14:34:44 2017
New Revision: 292435
URL: http://llvm.org/viewvc/llvm-project?rev=292435&view=rev
Log:
[Modules] Correct test comment from obsolete earlier version of code. NFC
Summary:
Code committed in rL290219 went through a few iterations; test wound up with
sta
Author: arpith
Date: Wed Jan 18 14:40:48 2017
New Revision: 292437
URL: http://llvm.org/viewvc/llvm-project?rev=292437&view=rev
Log:
[OpenMP] Support for the if-clause on the combined directive 'target parallel'.
The if-clause on the combined directive potentially applies to both the
'target' and
This revision was automatically updated to reflect the committed changes.
Closed by commit rL292436: [ASTReader] Add a DeserializationListener callback
for IMPORTED_MODULES (authored by graydon).
Changed prior to commit:
https://reviews.llvm.org/D28779?vs=84594&id=84881#toc
Repository:
rL LL
Author: graydon
Date: Wed Jan 18 14:36:59 2017
New Revision: 292436
URL: http://llvm.org/viewvc/llvm-project?rev=292436&view=rev
Log:
[ASTReader] Add a DeserializationListener callback for IMPORTED_MODULES
Summary:
Add a callback from ASTReader to DeserializationListener when the former
reads an
This revision was automatically updated to reflect the committed changes.
Closed by commit rL292435: [Modules] Correct test comment from obsolete earlier
version of code. NFC (authored by graydon).
Changed prior to commit:
https://reviews.llvm.org/D28790?vs=84616&id=84879#toc
Repository:
rL
efriedma added a comment.
> There would be a big performance penalty for ISRs with callees that don't use
> VFP regs.
Sacrificing correctness for the sake of performance seems like a bad idea...
especially given that the optimizer can insert calls to memcpy where they
didn't originally exist i
Author: stl_msft
Date: Wed Jan 18 14:10:25 2017
New Revision: 292434
URL: http://llvm.org/viewvc/llvm-project?rev=292434&view=rev
Log:
[libcxx] [test] Fix comment typos, strip trailing whitespace.
No functional change, no code review.
Modified:
libcxx/trunk/test/std/containers/associative/ma
Author: stl_msft
Date: Wed Jan 18 14:09:56 2017
New Revision: 292432
URL: http://llvm.org/viewvc/llvm-project?rev=292432&view=rev
Log:
[libcxx] [test] Fix MSVC warnings C4127 and C6326 about constants.
MSVC has compiler warnings C4127 "conditional expression is constant" (enabled
by /W4) and C632
vsk added a comment.
Ah ok, so it sounds like a better approach would be to split the missing record
message into a separate off-by-default warning. I don't have the time to update
this diff this week, but will shoot for the next.
https://reviews.llvm.org/D28867
jroelofs added a comment.
In https://reviews.llvm.org/D28820#649738, @jroelofs wrote:
> In https://reviews.llvm.org/D28820#649726, @efriedma wrote:
>
> > Why can't the compiler handle this case itself transparently? According to
> > your description, the interrupt calling convention is differen
timshen marked an inline comment as done.
timshen added a comment.
Friendly ping. :)
We still have internal test failures that this patch (and the next one) fixes,
and I think this is the last "hard to review" patch in the APFloat refactoring.
https://reviews.llvm.org/D27872
___
jroelofs added a comment.
In https://reviews.llvm.org/D28820#649726, @efriedma wrote:
> Why can't the compiler handle this case itself transparently? According to
> your description, the interrupt calling convention is different from the
> normal hard-float AAPCS calling convention: the VFP re
bader added inline comments.
Comment at: docs/UsersManual.rst:2065
+
+ $ clang -cc1 -triple spir64-unknown-unknown -cl-ext=-cl_khr_fp64 test.cl
+
pekka.jaaskelainen wrote:
> Anastasia wrote:
> > pekka.jaaskelainen wrote:
> > > Is this correct? I cannot make i
xur added a comment.
This change gonna hide the missing profile for a function permanently if there
is no mismatch. Like, if a user adds a new function without changing existing
functions, he will never get a warning if using the old profiles.
In LLVM IR level PGO, we also find the missing prof
This revision was automatically updated to reflect the committed changes.
Closed by commit rL292428: [OpenMP] Codegen for the 'target parallel' directive
on the NVPTX device. (authored by arpith).
Changed prior to commit:
https://reviews.llvm.org/D28755?vs=84512&id=84870#toc
Repository:
rL L
Author: arpith
Date: Wed Jan 18 13:35:00 2017
New Revision: 292428
URL: http://llvm.org/viewvc/llvm-project?rev=292428&view=rev
Log:
[OpenMP] Codegen for the 'target parallel' directive on the NVPTX device.
This patch adds codegen for the 'target parallel' directive on the NVPTX
device. We term
efriedma added a comment.
Why can't the compiler handle this case itself transparently? According to
your description, the interrupt calling convention is different from the normal
hard-float AAPCS calling convention: the VFP registers are all callee-save.
The compiler knows this; it should b
bruno accepted this revision.
bruno added a comment.
This revision is now accepted and ready to land.
LGTM!
Comment at: lib/Sema/SemaExpr.cpp:11523
// being assigned to.
if (Opc == BO_Assign && pty->getKind() == BuiltinType::Overload) {
+ if (getLangOpts().CPlusP
Author: rsmith
Date: Wed Jan 18 13:19:22 2017
New Revision: 292426
URL: http://llvm.org/viewvc/llvm-project?rev=292426&view=rev
Log:
PR9551: Implement DR1004 (http://wg21.link/cwg1004).
This rule permits the injected-class-name of a class template to be used as
both a template type argument and a
vsk created this revision.
Clang warns that a profile is out-of-date if it can't find a profile
record for any function in a TU. This warning is now noisy because llvm
can dead-strip functions with profiling instrumentation.
Only emit the out-of-date warning if there is an actual record mismatch.
On 1/18/17 11:26 AM, Richard Smith via cfe-commits wrote:
On 18 January 2017 at 10:12, Jonathan Roelofs via cfe-commits
mailto:cfe-commits@lists.llvm.org>> wrote:
Author: jroelofs
Date: Wed Jan 18 12:12:39 2017
New Revision: 292418
URL: http://llvm.org/viewvc/llvm-project?rev=
Author: arpith
Date: Wed Jan 18 12:18:53 2017
New Revision: 292419
URL: http://llvm.org/viewvc/llvm-project?rev=292419&view=rev
Log:
[OpenMP] Codegen support for 'target parallel' on the host.
This patch adds support for codegen of 'target parallel' on the host.
It is also the first combined dire
On 18 January 2017 at 10:12, Jonathan Roelofs via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> Author: jroelofs
> Date: Wed Jan 18 12:12:39 2017
> New Revision: 292418
>
> URL: http://llvm.org/viewvc/llvm-project?rev=292418&view=rev
> Log:
> Revert r286788
>
> The Itanium ABI [1] specifies t
Author: jroelofs
Date: Wed Jan 18 12:12:39 2017
New Revision: 292418
URL: http://llvm.org/viewvc/llvm-project?rev=292418&view=rev
Log:
Revert r286788
The Itanium ABI [1] specifies that __cxa_demangle accept either:
1) symbol names, which start with "_Z"
2) type manglings, which do not star
LegalizeAdulthood added a comment.
Don't do this without introducing an option to turn it off.
https://reviews.llvm.org/D28667
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
This revision was automatically updated to reflect the committed changes.
ehsan marked an inline comment as done.
Closed by commit rL292415: [clang-tidy] Add -extra-arg and -extra-arg-before to
run-clang-tidy.py (authored by ehsan).
Changed prior to commit:
https://reviews.llvm.org/D28334?vs=83
Author: ehsan
Date: Wed Jan 18 11:49:35 2017
New Revision: 292415
URL: http://llvm.org/viewvc/llvm-project?rev=292415&view=rev
Log:
[clang-tidy] Add -extra-arg and -extra-arg-before to run-clang-tidy.py
Summary:
These flags allow specifying extra arguments to the tool's command
line which don't a
ehsan marked an inline comment as done.
ehsan added inline comments.
Comment at: clang-tidy/tool/run-clang-tidy.py:80
+ for arg in extra_arg:
+ start.append('-extra-arg=%s' % arg[0])
+ for arg in extra_arg_before:
alexfh wrote:
> Why arg[0] and not just ar
Oh, remembered I had one other question/idea:
Tangentially related to the question about non-inline functions in headers:
Currently .pcm files have the 'interesting decls' list. Are there any
interesting decls once modules codegen is in use? It seems mostly the
interesting decls are things like n
On Wed, Jan 18, 2017 at 4:12 AM Hal Finkel via Phabricator <
revi...@reviews.llvm.org> wrote:
> hfinkel added a comment.
>
> Can you provide a high-level description of what you're trying to
> accomplish and the usage model?
>
Oh, for sure - sorry for the assumption/missing info. Stuff that's bee
Author: d0k
Date: Wed Jan 18 10:22:58 2017
New Revision: 292405
URL: http://llvm.org/viewvc/llvm-project?rev=292405&view=rev
Log:
[include-fixer] Don't return a correction if the header insertion failed.
This is could happen in cases involving macros and we don't want to
return an invalid fixit f
Author: d0k
Date: Wed Jan 18 10:25:48 2017
New Revision: 292406
URL: http://llvm.org/viewvc/llvm-project?rev=292406&view=rev
Log:
[ASTUnit] Reset diag state when creating the ASTUnit.
A client could call this with a dirty diagnostic engine, don't crash.
Modified:
cfe/trunk/lib/Frontend/ASTUn
pekka.jaaskelainen added inline comments.
Comment at: docs/UsersManual.rst:2065
+
+ $ clang -cc1 -triple spir64-unknown-unknown -cl-ext=-cl_khr_fp64 test.cl
+
Anastasia wrote:
> pekka.jaaskelainen wrote:
> > Is this correct? I cannot make it work:
> >
> > ``
arphaman added inline comments.
Comment at: lib/Sema/SemaCodeComplete.cpp:961
+// then incorrectly applied to the target declaration. This can be avoided
+// by resetting the declaration that's being hidden.
+if (Hiding && isa(Hiding))
ahatanak wrote:
arphaman updated this revision to Diff 84842.
arphaman marked an inline comment as done.
arphaman added a comment.
Avoid the hidden declaration in `VisibleDeclsRecord::checkHidden` instead of
the decl consumer in code-completion
Repository:
rL LLVM
https://reviews.llvm.org/D28514
Files:
l
mgorny created this revision.
Herald added a subscriber: dberris.
Use approximate comparison between the result of __divsc3()
and the canonical value calculated, to allow the possible difference of
1 representable value resulting from optimization.
For example, the value of (0.01+j0.01) /
arphaman added a comment.
Ping
Repository:
rL LLVM
https://reviews.llvm.org/D25213
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: d0k
Date: Wed Jan 18 09:50:26 2017
New Revision: 292402
URL: http://llvm.org/viewvc/llvm-project?rev=292402&view=rev
Log:
[Basic] Remove source manager references from diag state points.
This is just wasted space, we don't support state points from multiple
source managers. Validate that
Author: arpith
Date: Wed Jan 18 09:36:05 2017
New Revision: 292400
URL: http://llvm.org/viewvc/llvm-project?rev=292400&view=rev
Log:
Revert r292374 to debug Windows buildbot failure.
Removed:
cfe/trunk/test/OpenMP/target_parallel_codegen.cpp
cfe/trunk/test/OpenMP/target_parallel_codegen_
Author: hans
Date: Wed Jan 18 09:32:43 2017
New Revision: 292398
URL: http://llvm.org/viewvc/llvm-project?rev=292398&view=rev
Log:
Creating release directory for release_400.
Added:
libunwind/tags/RELEASE_400/
___
cfe-commits mailing list
cfe-commi
Author: hans
Date: Wed Jan 18 09:32:46 2017
New Revision: 292399
URL: http://llvm.org/viewvc/llvm-project?rev=292399&view=rev
Log:
Creating release candidate rc1 from release_400 branch
Added:
libunwind/tags/RELEASE_400/rc1/
- copied from r292398, libunwind/branches/release_40/
___
Author: hans
Date: Wed Jan 18 09:32:14 2017
New Revision: 292385
URL: http://llvm.org/viewvc/llvm-project?rev=292385&view=rev
Log:
Creating release candidate rc1 from release_400 branch
Added:
libcxx/tags/RELEASE_400/rc1/ (props changed)
- copied from r292384, libcxx/branches/release_
Author: hans
Date: Wed Jan 18 09:32:18 2017
New Revision: 292387
URL: http://llvm.org/viewvc/llvm-project?rev=292387&view=rev
Log:
Creating release candidate rc1 from release_400 branch
Added:
libcxxabi/tags/RELEASE_400/rc1/
- copied from r292386, libcxxabi/branches/release_40/
___
Author: hans
Date: Wed Jan 18 09:32:16 2017
New Revision: 292386
URL: http://llvm.org/viewvc/llvm-project?rev=292386&view=rev
Log:
Creating release directory for release_400.
Added:
libcxxabi/tags/RELEASE_400/
___
cfe-commits mailing list
cfe-commi
Author: hans
Date: Wed Jan 18 09:32:11 2017
New Revision: 292384
URL: http://llvm.org/viewvc/llvm-project?rev=292384&view=rev
Log:
Creating release directory for release_400.
Added:
libcxx/tags/RELEASE_400/
___
cfe-commits mailing list
cfe-commits@
jroelofs closed this revision.
jroelofs added a comment.
r292375
https://reviews.llvm.org/D28820
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: jroelofs
Date: Wed Jan 18 09:31:11 2017
New Revision: 292375
URL: http://llvm.org/viewvc/llvm-project?rev=292375&view=rev
Log:
Warn when calling a non interrupt function from an interrupt on ARM
The idea for this originated from a really tricky bug: ISRs on ARM don't
automatically save of
jroelofs added a comment.
In https://reviews.llvm.org/D28820#649384, @rengolin wrote:
> Seems like a very specific corner case on ARM, but is that attribute
> guaranteed to be ARM-only?
>
> If so, LGTM as is. If not, avoid mentioning "VFP" on the error message.
Yeah, the attribute is parsed in
malcolm.parsons added a comment.
In https://reviews.llvm.org/D28850#649462, @aaron.ballman wrote:
> LGTM, though I freely admit that I am no doxygen expert (from reading the
> docs, this looks like the sensible approach though).
I couldn't find a code owner for docs to ask for review.
There a
This revision was automatically updated to reflect the committed changes.
Closed by commit rL292374: [OpenMP] Codegen support for 'target parallel' on
the host. (authored by arpith).
Changed prior to commit:
https://reviews.llvm.org/D28753?vs=84689&id=84832#toc
Repository:
rL LLVM
https://r
Author: arpith
Date: Wed Jan 18 09:14:52 2017
New Revision: 292374
URL: http://llvm.org/viewvc/llvm-project?rev=292374&view=rev
Log:
[OpenMP] Codegen support for 'target parallel' on the host.
This patch adds support for codegen of 'target parallel' on the host.
It is also the first combined dire
ABataev accepted this revision.
ABataev added a comment.
This revision is now accepted and ready to land.
LG
https://reviews.llvm.org/D28781
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-c
svenvh created this revision.
Prior to OpenCL 2.0, image3d_t can only be used with the write_only
access qualifier when the cl_khr_3d_image_writes extension is enabled.
Clang did not diagnose write_only image3d arguments when the extension
was disabled.
Guard uses of write_only image3d in the Ope
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
LGTM, though I freely admit that I am no doxygen expert (from reading the docs,
this looks like the sensible approach though).
https://reviews.llvm.org/D28850
___
davidb added a comment.
ping
Repository:
rL LLVM
https://reviews.llvm.org/D27985
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Armpit, fine, leave it as is
Best regards,
Alexey Bataev
> 18 янв. 2017 г., в 15:50, Arpith Jacob via Phabricator
> написал(а):
>
> arpith-jacob added inline comments.
>
>
>
> Comment at: lib/Sema/SemaOpenMP.cpp:1933-1937
> + StmtResult SR = S;
> + int ThisCaptureLevel =
>
rengolin accepted this revision.
rengolin added a comment.
This revision is now accepted and ready to land.
Seems like a very specific corner case on ARM, but is that attribute guaranteed
to be ARM-only?
If so, LGTM as is. If not, avoid mentioning "VFP" on the error message.
https://reviews.ll
arpith-jacob added inline comments.
Comment at: lib/Sema/SemaOpenMP.cpp:1933-1937
+ StmtResult SR = S;
+ int ThisCaptureLevel =
+ getOpenMPCaptureLevels(DSAStack->getCurrentDirective());
+ while (--ThisCaptureLevel >= 0)
+SR = ActOnCapturedRegionEnd(SR.get());
---
ABataev accepted this revision.
ABataev added a comment.
This revision is now accepted and ready to land.
LG with nits
Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:848-860
+const StringRef outlinedHelperName, const RegionCodeGenTy &CodeGen) {
assert(ThreadIDVar->getType()
hfinkel added a comment.
Can you provide a high-level description of what you're trying to accomplish
and the usage model?
https://reviews.llvm.org/D28845
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman
arpith-jacob updated this revision to Diff 84816.
arpith-jacob added a comment.
Inherit from OMPLexical scope with an added argument to reduce code duplication.
https://reviews.llvm.org/D28781
Files:
include/clang/AST/OpenMPClause.h
include/clang/AST/RecursiveASTVisitor.h
lib/AST/OpenMPCl
malcolm.parsons created this revision.
Docs for clang::Decl and clang::TemplateSpecializationType have
not been generated since LLVM_ALIGNAS was added to them.
Tell Doxygen to expand LLVM_ALIGNAS to nothing as described at
https://www.stack.nl/~dimitri/doxygen/manual/preprocessing.html
https://
leanil updated this revision to Diff 84805.
leanil added a comment.
Add test cases for the new functionality.
These should not produce warnings, because their raw string replacement would
be longer.
https://reviews.llvm.org/D28667
Files:
clang-tidy/modernize/RawStringLiteralCheck.cpp
clang
1 - 100 of 106 matches
Mail list logo
