Today

arsenm created D35805: RA: Replace asserts related to empty live intervals.
Mon, Jul 24, 9:38 AM
grosser committed rL308895: Move MemoryAccess::getPwAff to isl++.
Move MemoryAccess::getPwAff to isl++
Mon, Jul 24, 9:37 AM
davidxl added a comment to D35804: [BPI] Detect branches in loops that make themselves not taken.

Why not putting this change into BranchProbablityInfo.cpp, so that other components can benefit from the more precise BP info?

Mon, Jul 24, 9:34 AM
aditya_nandakumar added a comment to D35516: [GISel]: Split buildSources part of buildInstr separately.

@dsanders - the main goal was to avoid the compiler warning of having to create a zero sized array. Now with the splitting of arguments, we can also accept MachineOperands which can directly be added to the instruction.

Mon, Jul 24, 9:33 AM
rnk committed rL308894: Add missing case to switch.
Add missing case to switch
Mon, Jul 24, 9:32 AM
aditya_nandakumar updated the diff for D35516: [GISel]: Split buildSources part of buildInstr separately.

Add the ability to accept machine operands to the buildInstr method so it will directly call MIB.add(MO) on that.

Mon, Jul 24, 9:32 AM
rnk accepted D35757: Work around an MSVC2017 update 3 codegen bug..

lgtm

Mon, Jul 24, 9:25 AM
grosser committed rL308892: Move ScopArrayInfo::getFromAccessFunction and getFromId to isl++.
Move ScopArrayInfo::getFromAccessFunction and getFromId to isl++
Mon, Jul 24, 9:24 AM
grosser committed rL308893: Move MemoryAccess::MemoryAccess to isl++.
Move MemoryAccess::MemoryAccess to isl++
Mon, Jul 24, 9:24 AM
john.brawn created D35804: [BPI] Detect branches in loops that make themselves not taken.
Mon, Jul 24, 9:24 AM
erik.pilkington accepted D35726: unguarded availability: add a fixit for the "annotate '...' with an availability attribute to silence" note.

LGTM, thanks for working on this!

Mon, Jul 24, 9:24 AM
d0k committed rL308891: [CodeGenPrepare] Cut off FindAllMemoryUses if there are too many uses..
[CodeGenPrepare] Cut off FindAllMemoryUses if there are too many uses.
Mon, Jul 24, 9:18 AM
rnk committed rL308890: [codeview] Emit 'D' as the cv source language for D code.
[codeview] Emit 'D' as the cv source language for D code
Mon, Jul 24, 9:17 AM
rnk committed rL308889: Format some case labels and shrink an anonymous namespace NFC.
Format some case labels and shrink an anonymous namespace NFC
Mon, Jul 24, 9:16 AM
grimar updated the summary of D35803: [ELF] - Fix calculation of memory region offset..
Mon, Jul 24, 9:12 AM
grimar created D35803: [ELF] - Fix calculation of memory region offset..
Mon, Jul 24, 9:12 AM
alexfh added a comment to D35787: [clang-tidy] Ignore vector<bool> in inefficient-vector-operation..

Should we look at whether the size is statically known?

Mon, Jul 24, 9:12 AM
alexfh added a comment to D35787: [clang-tidy] Ignore vector<bool> in inefficient-vector-operation..

Should we look at whether the size is statically known?

Mon, Jul 24, 9:12 AM
fjricci planned changes to D35798: [compiler-rt] Allow libignore to handle modules with multiple executable sections.

I think the need for this may actually reveal a deeper issue. Only the address range for the first executable section will be added to the suppression. The two ways around that would be to 1) allow the suppression to take multiple ranges or 2) don't use sections in procmaps for executable segments (lsan doesn't care about executable segments anyway)

Mon, Jul 24, 9:10 AM
alexfh requested changes to D34654: Allow passing a regex for headers to exclude from clang-tidy.
Mon, Jul 24, 9:07 AM · Restricted Project
aditya_nandakumar added a comment to D35332: [GISel]: More flexibility while building I/FCmps with MachineIRBuilder.

Hi Daniel - having these methods would let me use them in https://reviews.llvm.org/D35737. Right now they are not related.

Mon, Jul 24, 9:07 AM
alexfh requested changes to D24892: [clang-tidy] Add option "LiteralInitializers" to cppcoreguidelines-pro-type-member-init.

A few more nits.

Mon, Jul 24, 9:07 AM
DavidKreitzer added a comment to D34601: [X86][LLVM]Expanding Supports lowerInterleavedStore() in X86InterleavedAccess..

Thanks, Michael. This LGTM pending @RKSimon's re-review. But can you please update your sources so that we can see how you merged with https://reviews.llvm.org/D35638, since that will cause conflicts?

Mon, Jul 24, 9:05 AM
bcain created D35802: XFAIL tests on SLES11.
Mon, Jul 24, 9:04 AM
ibiryukov committed rL308888: NFC. Fixed typos in the comments..
NFC. Fixed typos in the comments.
Mon, Jul 24, 9:03 AM
rafael added a comment to D34781: Introduce a MCReloc class.

Ping, Gerolf, are you OK with this?

Mon, Jul 24, 9:03 AM
rafael updated the diff for D34988: Reduce duplication in pcrel reloc creation.

Rebased.

Mon, Jul 24, 8:59 AM
alexfh requested changes to D33829: [clang-tidy] avoid reserved names check.
Mon, Jul 24, 8:59 AM · Restricted Project
rafael updated the diff for D34781: Introduce a MCReloc class.

Rebased.

Mon, Jul 24, 8:58 AM
majnemer accepted D34972: [CodeGen] Propagate dllexport to thunks.

LGTM

Mon, Jul 24, 8:58 AM
mehdi_amini added a comment to D33900: Print registered targets in clang's version information.

Here is the current output:

Mon, Jul 24, 8:53 AM
alexfh accepted D35790: [clang-tidy] Handle incomplete types in bugprone-undefined-memory-manipulation check..

LG. Thank you for the fix!

Mon, Jul 24, 8:50 AM · Restricted Project
mehdi_amini added a reviewer for D33900: Print registered targets in clang's version information: hans.
Mon, Jul 24, 8:50 AM
inouehrs created D35801: [PowerPC] enable optimizeCompareInstr for branch with static branch hint.
Mon, Jul 24, 8:43 AM
Meinersbur added a comment to D35754: [ForwardOpTree] Introduce the -polly-optree pass..

Thank you. This is really nice idea. Do you have some cost modeling for when to allow this forwarding?

Mon, Jul 24, 8:40 AM · Restricted Project
thakis added a comment to D33900: Print registered targets in clang's version information.

Sorry, I just noticed this weeks later. Why are we adding this to --version instead of adding some new flag for printing this? When I pass --version, I'm usually interested in clang's version and don't need a screenful of other information below it (which makes the output I do care about scroll off the screen).

Mon, Jul 24, 8:37 AM
mcrosier added a comment to D35577: Add -flookup-tables and -fno-lookup-tables flags.

"Should this just be part of the tuning for the hexagon backend and not options at all"

I don't think we need separate options to control jump tables and lookup tables: one option to enable/disable the use of memory tables would be sufficient.
Mon, Jul 24, 8:37 AM
davide accepted D35115: X86 Asm should produce error messages instead of assertions when it's possible.

LGTM but please test with assertions on :)

Mon, Jul 24, 8:37 AM
Meinersbur committed rL308887: [ForwardOpTree] Properly indent enumeration in comment. NFC..
[ForwardOpTree] Properly indent enumeration in comment. NFC.
Mon, Jul 24, 8:34 AM
Meinersbur committed rL308886: [ForwardOpTree] Rename FD_CanForward to FD_CanForwardLeaf. NFC..
[ForwardOpTree] Rename FD_CanForward to FD_CanForwardLeaf. NFC.
Mon, Jul 24, 8:34 AM