jtbandes updated this revision to Diff 151858.
jtbandes marked an inline comment as done.
jtbandes added a comment.
Added an error when language standard doesn't support digraphs.
Still keeping `-fdigraphs` as a cc1 option because then I can distinguish
explicitly-enabled/disabled from the absen
SjoerdMeijer accepted this revision.
SjoerdMeijer added a comment.
This revision is now accepted and ready to land.
Looks OK to me.
Comment at: test/CodeGen/spir-half-type.cpp:89
+
+_Float16 fadd() {
+ _Float16 a = 1.0f16;
Nit: let one of these functions take
sammccall added inline comments.
Comment at: clangd/ClangdServer.cpp:119
+ auto FS = FSProvider.getFileSystem();
+ auto Status = FS->status(RootPath);
+ if (!Status)
why validate it?
Comment at: clangd/FindSymbols.h:30
/// \p Limit limits t
yvvan created this revision.
yvvan added reviewers: ilya-biryukov, arphaman, erikjv.
In case two translation units are created for the same file - reuse preamble
data to reduce memory and save time on extra preamble generation.
https://reviews.llvm.org/D48314
Files:
include/clang/Frontend/AS
Szelethus updated this revision to Diff 151866.
Szelethus added a comment.
In https://reviews.llvm.org/D48285#1136059, @NoQ wrote:
> Also, great, and can i has tests?^^
>
> Like a simple code snippet with two `// RUN: ... -analyzer-output=text` lines
> and different expected-warnings/notes under
Szelethus added a comment.
In https://reviews.llvm.org/D48285#1135480, @xazax.hun wrote:
> I wonder if this could be done when plist is emitted generally, independent
> of any checks.
Well, there's `-analyzer-config notes-as-events=true`. By the time plist files
are generated, every warning i
baloghadamsoftware added inline comments.
Comment at: test/clang-tidy/bugprone-exception-escape.cpp:178
+void indirect_implicit() noexcept {
+ // CHECK-MESSAGES: :[[@LINE-1]]:6: warning: an exception may be thrown in
function 'indirect_implicit' which should not throw exception
yvvan created this revision.
yvvan added reviewers: ilya-biryukov, arphaman, erikjv.
Append ASTUnit cache with global completions cache. In case of multiple
translation units for the same file reduces the memory consumption and save
completions cache generation time.
https://reviews.llvm.org/D
Author: szelethus
Date: Tue Jun 19 01:35:02 2018
New Revision: 335030
URL: http://llvm.org/viewvc/llvm-project?rev=335030&view=rev
Log:
[analyzer] Made a buildbot happy.
Since `isPrimitiveType` was only used in an assert, a builbot with `-Werror`
and no asserts enabled failed to build it as it wa
ioeric added inline comments.
Comment at: clangd/ClangdServer.cpp:119
+ auto FS = FSProvider.getFileSystem();
+ auto Status = FS->status(RootPath);
+ if (!Status)
sammccall wrote:
> why validate it?
This is the current behavior, except `llvm::sys::fs::is_direc
ioeric updated this revision to Diff 151869.
ioeric marked 3 inline comments as done.
ioeric added a comment.
- Address review comments
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D48290
Files:
clangd/ClangdServer.cpp
clangd/FindSymbols.cpp
clangd/FindSymbols.h
unitte
baloghadamsoftware updated this revision to Diff 151876.
baloghadamsoftware added a comment.
Typo fixed.
https://reviews.llvm.org/D33537
Files:
clang-tidy/bugprone/BugproneTidyModule.cpp
clang-tidy/bugprone/CMakeLists.txt
clang-tidy/bugprone/ExceptionEscapeCheck.cpp
clang-tidy/bugprone/
sammccall accepted this revision.
sammccall added inline comments.
This revision is now accepted and ready to land.
Comment at: unittests/clangd/TestFS.cpp:93
return URI(Scheme, /*Authority=*/"",
- llvm::sys::path::convert_to_slash(Body));
+ Strin
ioeric updated this revision to Diff 151877.
ioeric added a comment.
- Require '/' in front of unittest: body
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D48290
Files:
clangd/ClangdServer.cpp
clangd/FindSymbols.cpp
clangd/FindSymbols.h
unittests/clangd/FindSymbolsTest
Author: ioeric
Date: Tue Jun 19 02:33:53 2018
New Revision: 335035
URL: http://llvm.org/viewvc/llvm-project?rev=335035&view=rev
Log:
[clangd] Use workspace root path as hint path for resolving URIs in
workspace/symbol
Summary:
Some URI schemes require a hint path to be provided, and workspace ro
This revision was automatically updated to reflect the committed changes.
Closed by commit rL335035: [clangd] Use workspace root path as hint path for
resolving URIs in… (authored by ioeric, committed by ).
Herald added a subscriber: llvm-commits.
Repository:
rL LLVM
https://reviews.llvm.org/D
alexfh added a comment.
In https://reviews.llvm.org/D47670#1135497, @george.karpenkov wrote:
> @alexfh apologies, fixed.
Thank you1
Repository:
rC Clang
https://reviews.llvm.org/D47670
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
ht
Szelethus created this revision.
Szelethus added reviewers: NoQ, george.karpenkov, xazax.hun, rnkovacs, dkrupp.
Herald added subscribers: cfe-commits, mikhail.ramalho, a.sidorin, szepet,
whisperity.
After some thinking, I don't think this checker should support lambdas.
Reason 1.: While this is
Szelethus added a comment.
Thanks for the review!
I spend some time thinking about the support for lambda functions, and I start
to think that checking for lambda misuse shouldn't be the responsibility of
this checker. I created a new revision for that discussion, I wouldn't like to
abandon th
Higuoxing added a comment.
Sorry, It seems a little bit difficult for me to add a proper fix-it hint for
expressions in macros, because I cannot find the exact position of the last
char of the token on right hand side of operator. Any suggestion?
Actually, in gcc, it will emit warning for every
martong accepted this revision.
martong added a comment.
This revision is now accepted and ready to land.
Herald added a subscriber: rnkovacs.
LGTM, just found some minor things.
Comment at: unittests/AST/ASTImporterTest.cpp:174
TranslationUnitDecl *TUDecl = nullptr;
+
GBuella updated this revision to Diff 151884.
GBuella added a comment.
Added `__builtin_ia32_cmpsd_mask` & `__builtin_ia32_cmpss_mask`.
https://reviews.llvm.org/D45616
Files:
lib/CodeGen/CGBuiltin.cpp
test/CodeGen/avx-builtins.c
test/CodeGen/avx-cmp-builtins.c
test/CodeGen/avx2-builtins
Szelethus added a comment.
> [...]lambda misuse in not really an uninitialized value problem.
I guess you can make the argument that it its. Even then, in my opinion this
checker is overkill for lambdas.
- If the captured variable has a non-default constructor,
`UninitializedValueChecker` will
RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.
OK, if the llvm side tests are incoming I'm happy with this patch
https://reviews.llvm.org/D47401
___
cfe-commits mailing list
cfe-commits@list
erik.pilkington created this revision.
erik.pilkington added a reviewer: rsmith.
The constexpr evaluator was erroring out because these templates weren't
defined. Despite being used in a discarded statement, we still need to
constexpr evaluate them, which means that we need to instantiate them.
CarlosAlbertoEnciso added a comment.
@probinson Thanks very much for your review. I will address them.
https://reviews.llvm.org/D46190
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
CarlosAlbertoEnciso added a comment.
rsmith Thanks very much for your review. I will address them.
https://reviews.llvm.org/D46190
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ebevhan added a comment.
Just a couple more comments and then I think it looks good.
We can discuss the conversion and comparison issues in later patches.
Comment at: include/clang/AST/ASTContext.h:1951
+ unsigned char getFixedPointScale(const QualType &Ty) const;
+ unsigned
gramanas added a comment.
ping! should I commit this?
Repository:
rC Clang
https://reviews.llvm.org/D47720
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: aaronballman
Date: Tue Jun 19 07:36:04 2018
New Revision: 335048
URL: http://llvm.org/viewvc/llvm-project?rev=335048&view=rev
Log:
Implement semantic checking for __builtin_signbit.
r242675 changed the signature for the signbit builtin but did not introduce
proper semantic checking to en
aaron.ballman closed this revision.
aaron.ballman added a comment.
I've committed in r335048; if @rsmith has concerns, they can be address post
commit.
https://reviews.llvm.org/D47435
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://l
Author: aaronballman
Date: Tue Jun 19 07:53:20 2018
New Revision: 335049
URL: http://llvm.org/viewvc/llvm-project?rev=335049&view=rev
Log:
Reverting due to line ending changes; will reapply after addressing that.
Modified:
cfe/trunk/lib/Sema/SemaChecking.cpp
cfe/trunk/test/Sema/builtins.c
Author: aaronballman
Date: Tue Jun 19 07:59:11 2018
New Revision: 335050
URL: http://llvm.org/viewvc/llvm-project?rev=335050&view=rev
Log:
Implement semantic checking for __builtin_signbit.
r242675 changed the signature for the signbit builtin but did not introduce
proper semantic checking to en
martong accepted this revision.
martong added a comment.
This revision is now accepted and ready to land.
This patch is really useful and LGTM!
Just found some minor things.
Comment at: lib/AST/ASTImporter.cpp:7058
+const SrcMgr::ExpansionInfo &FromEx = FromSLoc.getExpansio
leonardchan marked an inline comment as done.
leonardchan added inline comments.
Comment at: lib/Sema/SemaExpr.cpp:1248
+ bool RHSFixed = RHSType->isFixedPointType();
+
+ if (LHSFixed && RHSFixed) {
ebevhan wrote:
> leonardchan wrote:
> > leonardchan wrote:
> >
leonardchan updated this revision to Diff 151919.
leonardchan marked 2 inline comments as done.
Repository:
rC Clang
https://reviews.llvm.org/D46915
Files:
include/clang-c/Index.h
include/clang/AST/ASTContext.h
include/clang/AST/Expr.h
include/clang/AST/RecursiveASTVisitor.h
include/
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
LGTM, thank you for this!
Repository:
rC Clang
https://reviews.llvm.org/D48100
___
cfe-commits mailing list
cfe-commits@lists.ll
martong requested review of this revision.
martong added a comment.
Ping.
Repository:
rC Clang
https://reviews.llvm.org/D47367
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
aaron.ballman added inline comments.
Comment at: lib/CodeGen/CGBuiltin.cpp:1250
+ if (Info.IsSigned) {
+unsigned HighBits = Info.StorageSize - Info.Offset - Info.Size;
+if (HighBits)
What happens if this overflows due to being < 0?
Reposito
ebevhan accepted this revision.
ebevhan added inline comments.
This revision is now accepted and ready to land.
Comment at: lib/Sema/SemaExpr.cpp:1248
+ bool RHSFixed = RHSType->isFixedPointType();
+
+ if (LHSFixed && RHSFixed) {
leonardchan wrote:
> ebevhan wr
Szelethus added inline comments.
Comment at: lib/StaticAnalyzer/Checkers/UninitializedObjectChecker.cpp:685
+
+ if (CXXParent && CXXParent->isLambda()) {
+CXXRecordDecl::capture_const_iterator CapturedVar =
george.karpenkov wrote:
> CXXParent is guaranteed t
Szelethus created this revision.
Szelethus added reviewers: NoQ, george.karpenkov, xazax.hun, rnkovacs.
Herald added subscribers: cfe-commits, mikhail.ramalho, a.sidorin, szepet,
whisperity.
Repository:
rC Clang
https://reviews.llvm.org/D48325
Files:
lib/StaticAnalyzer/Checkers/Uninitialize
craig.topper requested changes to this revision.
craig.topper added inline comments.
This revision now requires changes to proceed.
Comment at: test/CodeGen/avx-builtins.c:241
// CHECK-LABEL: test_mm_cmp_sd
- // CHECK: call <2 x double> @llvm.x86.sse2.cmp.sd(<2 x double> %{{.
smcgro updated this revision to Diff 151938.
smcgro added a comment.
Added a function parameter to one of the functions
https://reviews.llvm.org/D48188
Files:
lib/Basic/Targets/SPIR.h
test/CodeGen/spir-half-type.cpp
Index: test/CodeGen/spir-half-type.cpp
===
smcgro added a comment.
That's great @SjoerdMeijer , thanks for the review. I don't have commit access
to clang so if there are no more reviews then it would be great if someone
could commit this for me, thanks.
https://reviews.llvm.org/D48188
___
Szelethus updated this revision to Diff 151943.
Szelethus added a comment.
Accidently added the pointer `MemberPointer` objects twice, fixed that.
https://reviews.llvm.org/D48325
Files:
lib/StaticAnalyzer/Checkers/UninitializedObjectChecker.cpp
test/Analysis/cxx-uninitialized-object-ptr-ref
arphaman marked 3 inline comments as done.
arphaman added inline comments.
Comment at: lib/Frontend/InitHeaderSearch.cpp:374-377
+ IsBaseFound = AddGnuCPlusPlusIncludePaths("/usr/include/c++/4.2.1",
+"powerpc-apple-darwin10", "
arphaman updated this revision to Diff 151944.
arphaman marked 3 inline comments as done.
arphaman added a comment.
Addressed review comments
https://reviews.llvm.org/D48297
Files:
include/clang/Basic/DiagnosticFrontendKinds.td
include/clang/Lex/HeaderSearch.h
lib/Frontend/InitHeaderSearc
emmettneyman added inline comments.
Comment at: tools/clang-fuzzer/handle-llvm/CMakeLists.txt:5
+ handle_llvm.cpp
+ )
morehouse wrote:
> There's fewer libraries linked here than in `handle-cxx/` (not saying this is
> wrong, but it could be). Do you get link e
bruno accepted this revision.
bruno added a comment.
This revision is now accepted and ready to land.
LGTM
https://reviews.llvm.org/D48297
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-com
Szelethus abandoned this revision.
Szelethus added a comment.
Turns out I was very wrong on this one. Lambdas should be ignored as data
members, but not in general.
Sorry about the spam, I think I should've sit longer on this one. I'll submit
another patch tomorrow that will correctly explain m
Author: arphaman
Date: Tue Jun 19 10:56:03 2018
New Revision: 335063
URL: http://llvm.org/viewvc/llvm-project?rev=335063&view=rev
Log:
[Darwin] Add a warning for missing include path for libstdc++
Xcode 10 removes support for libstdc++, but the users just get a confusing
include not file warning
This revision was automatically updated to reflect the committed changes.
Closed by commit rL335063: [Darwin] Add a warning for missing include path for
libstdc++ (authored by arphaman, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm.org/D4
morehouse added inline comments.
Comment at: tools/clang-fuzzer/handle-llvm/CMakeLists.txt:5
+ handle_llvm.cpp
+ )
emmettneyman wrote:
> morehouse wrote:
> > There's fewer libraries linked here than in `handle-cxx/` (not saying this
> > is wrong, but it could
george.karpenkov added inline comments.
Comment at: lib/StaticAnalyzer/Checkers/UninitializedObjectChecker.cpp:685
+
+ if (CXXParent && CXXParent->isLambda()) {
+CXXRecordDecl::capture_const_iterator CapturedVar =
Szelethus wrote:
> george.karpenkov wrote:
>
rsmith accepted this revision.
rsmith added inline comments.
This revision is now accepted and ready to land.
Comment at: utils/hmaptool/hmaptool:1
+#!/usr/bin/env python
+
Have you checked this works with both python2 and python3?
Comment at:
emmettneyman added inline comments.
Comment at: tools/clang-fuzzer/handle-llvm/\:62
+ initializeScavengerTestPass(*Registry);
+
+}
morehouse wrote:
> Does this initialization need to happen every time the fuzzer generates a new
> input, or can we call this from
baloghadamsoftware added a comment.
I tested all parts of the Iterator Checkers, all tests passed.
https://reviews.llvm.org/D35110
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
guigu added inline comments.
Comment at: lib/Format/WhitespaceManager.cpp:678
// Indent with tabs only when there's at least one full tab.
-if (FirstTabWidth + Style.TabWidth <= Spaces) {
+if (Style.TabWidth <= Spaces) {
Spaces -= FirstTabWidth;
--
craig.topper added a comment.
Fast-isel tests were added for previous codegen in r335068 and updated for new
codegen in r335071.
One addtiional observation I didn't catch before. The epi32 and epu32 min/max
intrinsics were doing a 64-bit element extract as the final step previously
because the
Author: arphaman
Date: Tue Jun 19 12:43:07 2018
New Revision: 335073
URL: http://llvm.org/viewvc/llvm-project?rev=335073&view=rev
Log:
Revert r335063 as it causes bot failures
Removed:
cfe/trunk/test/Frontend/warning-stdlibcxx-darwin.cpp
Modified:
cfe/trunk/include/clang/Basic/DiagnosticF
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
In https://reviews.llvm.org/D48296#1136095, @thakis wrote:
> Thanks! I'd keep it DefaultIgnored: I'd be annoyed if I'd get these by
> default in C++98 mode.
Got it, for some reason I'd forgotten t
jdenny updated this revision to Diff 151979.
jdenny added a comment.
Rebased. Ping.
https://reviews.llvm.org/D46919
Files:
include/clang-c/Index.h
include/clang/AST/PrettyPrinter.h
lib/AST/DeclPrinter.cpp
lib/AST/TypePrinter.cpp
tools/c-index-test/c-index-test.c
Index: tools/c-index
emmettneyman added inline comments.
Comment at: tools/clang-fuzzer/handle-llvm/\:113
+
+ legacy::PassManager PM;
+ TargetLibraryInfoImpl TLII(Triple(M->getTargetTriple()));
morehouse wrote:
> Any reason not to use the newer PassManager?
Clang (`llc`) and the `l
Quuxplusone updated this revision to Diff 151983.
Quuxplusone added a comment.
Minor cosmetic changes.
Repository:
rCXX libc++
https://reviews.llvm.org/D47111
Files:
include/experimental/memory_resource
src/experimental/memory_resource.cpp
test/libcxx/experimental/memory/memory.resour
emmettneyman added inline comments.
Comment at: tools/clang-fuzzer/proto-to-llvm/loop_proto_to_llvm.cpp:71
+return val_var;
+ }
+}
morehouse wrote:
> Is it still possible for the protobuf to not have a constant, a binOp, or a
> varRef?
Right now, since the
Quuxplusone updated this revision to Diff 151984.
Quuxplusone added a comment.
Herald added a subscriber: mgrang.
Bugfix and shrink {un,}synchronized_pool_resource.
The old implementation was severely broken in two ways:
- It accidentally sometimes trusted the user's `bytes` and `align` argument
emmettneyman updated this revision to Diff 151986.
emmettneyman added a comment.
- made changes to handle_llvm.cpp in response to reviewer comments
Repository:
rC Clang
https://reviews.llvm.org/D48106
Files:
tools/clang-fuzzer/CMakeLists.txt
tools/clang-fuzzer/ExampleClangLLVMProtoFuzzer
Author: arphaman
Date: Tue Jun 19 15:47:53 2018
New Revision: 335081
URL: http://llvm.org/viewvc/llvm-project?rev=335081&view=rev
Log:
Recommit r335063: [Darwin] Add a warning for missing include path for libstdc++
The recommit ensures that the tests that failed on bots don't trigger the
warning
Author: nico
Date: Tue Jun 19 16:19:34 2018
New Revision: 335082
URL: http://llvm.org/viewvc/llvm-project?rev=335082&view=rev
Log:
clang-cl: Emit narrowing diag for initializer lists if -fmsc-version is at
least 1900 (i.e. MSVC2015).
Diagnostics for narrowing conversions in initializer lists are
thakis closed this revision.
thakis added a comment.
r335082, thanks!
https://reviews.llvm.org/D48296
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
morehouse added a comment.
If you haven't already, please apply for commit access:
https://llvm.org/docs/DeveloperPolicy.html#obtaining-commit-access.
That way you can land this after it's accepted.
Comment at: tools/clang-fuzzer/handle-llvm/handle_llvm.cpp:23
+#include "llv
This revision was automatically updated to reflect the committed changes.
Closed by commit rC335084: Append new attributes to the end of an
AttributeList. (authored by Meinersbur, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D48100?vs=151378&id=151998#toc
Repository:
rC
juliehockett created this revision.
juliehockett added reviewers: ioeric, lebedev.ri.
juliehockett added a project: clang-tools-extra.
The first reduce pass combines all duplicate info data into one representation
of that declaration info. This second pass iterates through the output of that
fir
vsapsai created this revision.
vsapsai added reviewers: mclow.lists, EricWF.
Herald added a subscriber: christof.
We already have a specialization that will use memcpy for construction
of trivial types from an iterator range like
std::vector(int *, int *);
But if we have const-ness mismatch li
emmettneyman updated this revision to Diff 152003.
emmettneyman added a comment.
- minor changes to improve readability and style
Repository:
rC Clang
https://reviews.llvm.org/D48106
Files:
tools/clang-fuzzer/CMakeLists.txt
tools/clang-fuzzer/ExampleClangLLVMProtoFuzzer.cpp
tools/clang
vsapsai added a comment.
Related change is https://reviews.llvm.org/D8109 For me the performance
improvement was a twice faster execution on a dirty benchmark that doesn't
exclude set up and tear down.
https://reviews.llvm.org/D48342
___
cfe-commi
Author: ctopper
Date: Tue Jun 19 17:31:39 2018
New Revision: 335086
URL: http://llvm.org/viewvc/llvm-project?rev=335086&view=rev
Log:
[X86] Undefine _mm512_mask_reduce_operator macro in avx512fintrin.h before
redefining it.
Modified:
cfe/trunk/lib/Headers/avx512fintrin.h
Modified: cfe/trunk
morehouse accepted this revision.
morehouse added inline comments.
This revision is now accepted and ready to land.
Comment at: tools/clang-fuzzer/handle-llvm/handle_llvm.cpp:50
+ errs() << "error: opt level must be between 0 and 3.\n";
+ std::exit(0);
+ }
emmettneyman updated this revision to Diff 152005.
emmettneyman added a comment.
- removed unnecessary comment and fixed exit statement
Repository:
rC Clang
https://reviews.llvm.org/D48106
Files:
tools/clang-fuzzer/CMakeLists.txt
tools/clang-fuzzer/ExampleClangLLVMProtoFuzzer.cpp
tools
morehouse added a comment.
Looks like `exit(0)` is still there.
Repository:
rC Clang
https://reviews.llvm.org/D48106
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
emmettneyman updated this revision to Diff 152006.
emmettneyman added a comment.
- actually fixed error statement
Repository:
rC Clang
https://reviews.llvm.org/D48106
Files:
tools/clang-fuzzer/CMakeLists.txt
tools/clang-fuzzer/ExampleClangLLVMProtoFuzzer.cpp
tools/clang-fuzzer/cxx_loop
emmettneyman added a comment.
In https://reviews.llvm.org/D48106#1137224, @morehouse wrote:
> Looks like `exit(0)` is still there.
oops, forgot to `:w`
Repository:
rC Clang
https://reviews.llvm.org/D48106
___
cfe-commits mailing list
cfe-commi
Author: dyung
Date: Tue Jun 19 18:05:09 2018
New Revision: 335088
URL: http://llvm.org/viewvc/llvm-project?rev=335088&view=rev
Log:
Add a test to verify the x86 intrinsic headers compile cleanly with no warnings
or errors.
Added:
cfe/trunk/test/Headers/x86-intrinsics-headers-clean.cpp
Adde
craig.topper created this revision.
craig.topper added a reviewer: RKSimon.
Herald added a subscriber: cfe-commits.
Similar to what was done to max/min recently.
These already reduced the vector width to 256 and 128 bit as we go unlike the
original max/min code.
Repository:
rC Clang
https:/
85 matches
Mail list logo
