Revision Log - llvm-project http://src.mouf.net/llvm-project/log http://src.mouf.net/llvm-project/rss https://pubsubhubbub.appspot.com/ 2019-10-22T15:57:12 SVN::Web daily 1 1901-01-01T00:00+00:00 /www/trunk/devmtg/2019-10/index.html @ 375507 http://src.mouf.net/llvm-project/revision?rev=375507 ]]> tbrethou 2019-10-22T15:57:12 /www/trunk/devmtg/2019-10/index.html @ 375506 http://src.mouf.net/llvm-project/revision?rev=375506 ]]> tbrethou 2019-10-22T15:49:17 /llvm/trunk/lib/Transforms/InstCombine/InstCombineInternal.h /llvm/trunk/lib/Transforms/InstCombine/InstCombineSelect.cpp /llvm/trunk/test/Transforms/InstCombine/sadd_sat.ll @ 375505 http://src.mouf.net/llvm-project/revision?rev=375505
This adds an instcombine matcher for code that attempts to perform signed
saturating arithmetic by casting to a higher type. Unsigned cases are already
matched, this adds extra matches for the more complex signed cases, which
involves matching the min(max(add a b)) nodes with proper extends to ensure
legality.

Differential Revision: https://reviews.llvm.org/D68651
]]>
dmgreen 2019-10-22T15:39:47
/libcxx/trunk/include/memory @ 375504 http://src.mouf.net/llvm-project/revision?rev=375504
Summary: This patch removes `shared_ptr::make_shared` as it is not part of the standard. This patch also adds __create_with_cntrl_block, which is a help function that can be used in std::allocate_shared and std::make_shared. This is the third patch (out of 4) from D66178.

Reviewers: EricWF, mclow.lists, ldionne

Subscribers: christof, dexonsmith, libcxx-commits

Tags: #libc

Differential Revision: https://reviews.llvm.org/D68805]]>
zoecarver 2019-10-22T15:16:49
/llvm/trunk/test/Transforms/InstCombine/sadd_sat.ll @ 375503 http://src.mouf.net/llvm-project/revision?rev=375503 dmgreen 2019-10-22T14:49:40 /llvm/trunk/lib/CodeGen/MIRParser/MIParser.cpp /llvm/trunk/test/CodeGen/MIR/Mips/setRegClassOrRegBank.mir /llvm/trunk/test/CodeGen/Mips/GlobalISel/legalizer/add_vec_builtin.mir @ 375502 http://src.mouf.net/llvm-project/revision?rev=375502
MachineRegisterInfo::createGenericVirtualRegister sets
RegClassOrRegBank to static_cast<RegisterBank *>(nullptr).
MIParser on the other hand doesn't. When we attempt to constrain
Register Class on such VReg, additional COPY is generated.
This way we avoid COPY instructions showing in test that have MIR
input while they are not present with llvm-ir input that was used
to create given MIR for a -run-pass test.

Differential Revision: https://reviews.llvm.org/D68946
]]>
petar.avramovic 2019-10-22T14:25:37
/llvm/trunk/lib/Target/Mips/MipsLegalizerInfo.cpp /llvm/trunk/lib/Target/Mips/MipsRegisterBankInfo.cpp /llvm/trunk/test/CodeGen/MIR/Mips/setRegClassOrRegBank.ll @ 375501 http://src.mouf.net/llvm-project/revision?rev=375501
Select vector G_ADD for MIPS32 with MSA. We have to set bank
for vector operands to fprb and selectImpl will do the rest.
__builtin_msa_addv_<format> will be transformed into G_ADD
in legalizeIntrinsic and selected in the same way.
__builtin_msa_addvi_<format> will be directly selected into
ADDVI_<format> in legalizeIntrinsic. MIR tests for it have
unnecessary additional copies. Capture current state of tests
with run-pass=legalizer with a test in test/CodeGen/MIR/Mips.

Differential Revision: https://reviews.llvm.org/D68984
]]>
petar.avramovic 2019-10-22T13:51:57
/llvm/trunk/lib/Transforms/IPO/FunctionImport.cpp @ 375500 http://src.mouf.net/llvm-project/revision?rev=375500 evgeny777 2019-10-22T12:57:23 /llvm/trunk/include/llvm/IR/Instructions.h /llvm/trunk/lib/AsmParser/LLParser.cpp /llvm/trunk/lib/Bitcode/Reader/BitcodeReader.cpp @ 375499 http://src.mouf.net/llvm-project/revision?rev=375499
Summary:
This is patch is part of a series to introduce an Alignment type.
See this thread for context: http://lists.llvm.org/pipermail/llvm-dev/2019-July/133851.html
See this patch for the introduction of the type: https://reviews.llvm.org/D64790

Reviewers: courbet

Subscribers: hiraditya, jfb, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D69303]]>
gchatelet 2019-10-22T12:55:32
/llvm/trunk/include/llvm/IR/Instructions.h /llvm/trunk/lib/AsmParser/LLParser.cpp /llvm/trunk/lib/Bitcode/Reader/BitcodeReader.cpp @ 375498 http://src.mouf.net/llvm-project/revision?rev=375498
Summary:
This is patch is part of a series to introduce an Alignment type.
See this thread for context: http://lists.llvm.org/pipermail/llvm-dev/2019-July/133851.html
See this patch for the introduction of the type: https://reviews.llvm.org/D64790

Reviewers: courbet

Subscribers: hiraditya, jfb, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D69302]]>
gchatelet 2019-10-22T12:35:55
/llvm/trunk/lib/Target/PowerPC/PPCReduceCRLogicals.cpp /llvm/trunk/lib/Target/PowerPC/PPCTargetMachine.cpp /llvm/trunk/test/CodeGen/PowerPC/2008-10-28-f128-i32.ll @ 375497 http://src.mouf.net/llvm-project/revision?rev=375497
This re-commits r375152 which was pulled in r375233 because it broke
the EXPENSIVE_CHECKS bot on Windows.

The reason for the failure was a bug in the pass that the commit turned
on by default. This patch fixes that bug and turns the pass back on.
This patch has been verified on the buildbot that originally failed
thanks to Simon Pilgrim.

Differential revision: https://reviews.llvm.org/D52431
]]>
nemanjai 2019-10-22T12:20:38
/llvm/trunk/include/llvm/IR/Attributes.h /llvm/trunk/lib/AsmParser/LLParser.cpp /llvm/trunk/lib/AsmParser/LLParser.h @ 375496 http://src.mouf.net/llvm-project/revision?rev=375496
Summary:
This is patch is part of a series to introduce an Alignment type.
See this thread for context: http://lists.llvm.org/pipermail/llvm-dev/2019-July/133851.html
See this patch for the introduction of the type: https://reviews.llvm.org/D64790

Reviewers: courbet

Subscribers: hiraditya, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D69300]]>
gchatelet 2019-10-22T11:57:52
/llvm/trunk/include/llvm/IR/Attributes.h /llvm/trunk/include/llvm/IR/Function.h /llvm/trunk/include/llvm/IR/InstrTypes.h @ 375495 http://src.mouf.net/llvm-project/revision?rev=375495
Summary:
This is patch is part of a series to introduce an Alignment type.
See this thread for context: http://lists.llvm.org/pipermail/llvm-dev/2019-July/133851.html
See this patch for the introduction of the type: https://reviews.llvm.org/D64790

Reviewers: courbet

Subscribers: jholewinski, hiraditya, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D69278]]>
gchatelet 2019-10-22T09:51:06
/meta/trunk/github-usernames.txt @ 375494 http://src.mouf.net/llvm-project/revision?rev=375494 evgeny777 2019-10-22T09:26:24 /llvm/trunk/lib/Transforms/IPO/FunctionImport.cpp /llvm/trunk/test/ThinLTO/X86/not-internalized.ll /llvm/trunk/test/ThinLTO/X86/weak_externals.ll @ 375493 http://src.mouf.net/llvm-project/revision?rev=375493
Differential revision: https://reviews.llvm.org/D69107
]]>
evgeny777 2019-10-22T09:24:12
/llvm/trunk/lib/DebugInfo/PDB/Native/NativeEnumInjectedSources.cpp @ 375492 http://src.mouf.net/llvm-project/revision?rev=375492
Currently injected-sources-native.test fails with "Expected<T>
value was in success state.
(Note: Expected<T> values in success mode must still be checked
prior to being destroyed)"
when llvm is compiled with LLVM_ENABLE_ABI_BREAKING_CHECKS in Release.

The problem is that getStringForID returns Expected<StringRef>
and Expected value must always be checked, even if it is in success state.
Checking with assert only helps in Debug and is wrong.

Differential revision: https://reviews.llvm.org/D69251]]>
grimar 2019-10-22T08:52:45
/cfe/trunk/unittests/Frontend/OutputStreamTest.cpp @ 375491 http://src.mouf.net/llvm-project/revision?rev=375491
Setting the output stream to nulls seems to work.]]>
d0k 2019-10-22T08:44:34
/compiler-rt/trunk/test/builtins/Unit/clear_cache_test.c @ 375490 http://src.mouf.net/llvm-project/revision?rev=375490
Within the last two weeks, the Builtins-*-sunos :: clear_cache_test.c started to FAIL
on Solaris. Running it under truss shows

mmap(0x00000000, 128, PROT_READ|PROT_WRITE|PROT_EXEC, MAP_PRIVATE|MAP_ANON, 0, 0) Err#22 EINVAL
_exit(1)

While there are several possible reasons mmap can return EINVAL on Solaris, it turns
out it's this one (from mmap(2)):

MAP_ANON was specified, but the file descriptor was not
-1.

And indeed even the Linux mmap(2) documents this as unportable:

MAP_ANONYMOUS
The mapping is not backed by any file; its contents are initial‐
ized to zero. The fd argument is ignored; however, some imple‐
mentations require fd to be -1 if MAP_ANONYMOUS (or MAP_ANON) is
specified, and portable applications should ensure this. The

This patch follows this advise. Tested on x86_64-pc-linux-gnu, amd64-pc-solaris2.11
and sparcv9-sun-solaris2.11.

Differential Revision: https://reviews.llvm.org/D68455
]]>
ro 2019-10-22T08:44:25
/cfe/trunk/unittests/Frontend/OutputStreamTest.cpp @ 375489 http://src.mouf.net/llvm-project/revision?rev=375489
This reverts commit 375488.]]>
d0k 2019-10-22T08:37:15
/cfe/trunk/unittests/Frontend/OutputStreamTest.cpp @ 375488 http://src.mouf.net/llvm-project/revision?rev=375488 d0k 2019-10-22T08:10:51