SuccessChanges

Summary

  1. [Hexagon] Improve casting of boolean HVX vectors to scalars (details)
  2. Fix LLP64 detection in SwapByteOrder.h (details)
  3. [InstCombine] Remove trivially empty ranges from end (details)
  4. [AVR] Don't adjust addresses by 2 for absolute values (details)
  5. [AVR] Fix private label prefix (details)
  6. [CVP] Add test for cmp of local phi; NFC (details)
  7. [InstCombine] Simplify DCE code; NFC (details)
  8. [TSan] Add CMake check for minimal SDK requirements on Darwin (details)
  9. [CVP] Simplify cmp of local phi node (details)
  10. [SimpleLoopUnswitch] Remove unnecessary include; NFC (details)
  11. [clang] fix error detection in consteval calls (details)
  12. [mlir][spirv] Add some folders for spv.LogicalAnd/spv.LogicalOr (details)
  13. [VectorCombine] add a debug flag to skip all transforms (details)
  14. [ms] Rename ParsingInlineAsm functions/variables to reflect MS-specificity. (details)
  15. [dsymutil] Avoid copying swiftinterfaces from the SDK into the dsym bundle (details)
  16. Fix for the test failues introduced in https://reviews.llvm.org/D74704. (details)
  17. [libc] Fix typo of namespace in round implementation. (details)
  18. [scudo][standalone] Various improvements wrt RSS (details)
  19. [mlir][spirv] Refactoring to avoid calling the same function twice (details)
  20. [llvm-objcopy] Enable --discard-all for MachO (details)
  21. [MC] Pull out a relaxFragment helper [NFC] (details)
  22. Update the ARC docs for non-trivial ownership in structs. (details)
  23. [DebugInfo][NFC] Remove handler with ErrorPolicy from DWARFContext. (details)
  24. [ASTMatchers] HasNameMatcher handles `extern "C"` (details)
  25. [AArch64][GlobalISel] Fixup <32b heterogeneous regbanks of G_PHIs just before selection. (details)
  26. [ObjectFileMachO] Fix a build error on embedded. (details)
  27. [X86] Add proper MachinePointerInfo to the loads/stores created for moving data between SSE and X87 in X86DAGToDAGISel::PreprocessISelDAG (details)
  28. [X86] Use correct MachineMemOperand for stack load in LowerFLT_ROUNDS_ (details)
  29. [X86] Use proper alignment for stack temporary and correct MachinePointerInfo for stack accesses in LowerUINT_TO_FP. (details)
  30. [X86] Query constant pool object alignment instead of hardcoding. (details)
  31. Visit previously unreachable nodes in the debug info metadata verifier. (details)
  32. Disable memory leak checking in a test to work around a bot failure (details)
  33. PR44978: Accept as an extension some cases where destructor name lookup (details)
Commit 7691790dfd1011d08f5468f63952d7690755aad4 by kparzysz
[Hexagon] Improve casting of boolean HVX vectors to scalars

- Mark memory access for bool vectors as disallowed in target lowering.
  This will prevent combining bitcasts of bool vectors with stores.
- Replace the actual bitcasting code with a faster version.
- Handle casting of v16i1 to i16.
The file was modifiedllvm/lib/Target/Hexagon/HexagonISelLowering.cpp (diff)
The file was modifiedllvm/test/CodeGen/Hexagon/store-vector-pred.ll (diff)
The file was modifiedllvm/test/CodeGen/Hexagon/hvx-bitcast-v64i1.ll (diff)
The file was modifiedllvm/test/CodeGen/Hexagon/autohvx/isel-hvx-pred-bitcast.ll (diff)
The file was modifiedllvm/lib/Target/Hexagon/HexagonISelLowering.h (diff)
The file was addedllvm/test/CodeGen/Hexagon/autohvx/isel-store-bitcast-v128i1.ll
The file was modifiedllvm/lib/Target/Hexagon/HexagonISelLoweringHVX.cpp (diff)
Commit 30dae38349cbe293bee0705b313dc747e53afd49 by rnk
Fix LLP64 detection in SwapByteOrder.h

MSVC does not define __LONG_MAX__, so we were just getting lucky in this
conditional:
  #if __LONG_MAX__ == __INT_MAX__

Undefined identifiers evaluate to zero in preprocessor conditionals, so
this became true, which happens to work for MSVC platforms.

Instead, use this pattern and let the compiler constant fold:
  return sizeof(long) == sizeof(int) ? SwapByteOrder_32((uint32_t)C)
                                     : SwapByteOrder_64((uint64_t)C);
The file was modifiedllvm/include/llvm/Support/SwapByteOrder.h (diff)
Commit 56f7de5baae3ae9a746ac8f5a8e32cfcaf7b4a6b by nikita.ppv
[InstCombine] Remove trivially empty ranges from end

InstCombine removes pairs of start+end intrinsics that don't
have anything in between them. Currently this is done by starting
at the start intrinsic and scanning forwards. This patch changes
it to start at the end intrinsic and scan backwards.

The motivation here is as follows: When we process the start
intrinsic, we have not yet looked at the following instructions,
which may still get folded/removed. If they do, we will only be
able to remove the start/end pair on the next iteration. When we
process the end intrinsic, all the instructions before it have
already been visited, and we don't run into this problem.

Differential Revision: https://reviews.llvm.org/D75011
The file was modifiedllvm/test/Transforms/InstCombine/lifetime.ll (diff)
The file was modifiedllvm/test/Transforms/InstCombine/vararg.ll (diff)
The file was modifiedllvm/lib/Transforms/InstCombine/InstCombineCalls.cpp (diff)
The file was modifiedllvm/lib/Transforms/InstCombine/InstCombineInternal.h (diff)
Commit 165f707f9d0fb227e30b9a57d8ceb27f55032f90 by aykevanlaethem
[AVR] Don't adjust addresses by 2 for absolute values

Adjusting by 2 breaks DWARF output. With this fix, programs start to
compile and produce valid DWARF output.

Differential Revision: https://reviews.llvm.org/D74213
The file was modifiedllvm/lib/Target/AVR/MCTargetDesc/AVRAsmBackend.cpp (diff)
The file was addedllvm/test/MC/AVR/relocations-abs.s
Commit 6afc3de42f8f65dfa2e51b0e81e95ef6427be9ac by aykevanlaethem
[AVR] Fix private label prefix

This is a small pet peeve from me. This change makes sure the AVR backend uses
the correct private label prefix (.L) so that private labels are hidden in
avr-objdump.

Example code:

    define i8 @foo(i1 %cond) {
      br i1 %cond, label %then, label %else
    then:
      ret i8 3
    else:
      ret i8 5
    }

When compiling this:
  llc -march=avr -filetype=obj -o test.o test.ll
and then dumping it:
  avr-objdump -d test.o
You would previously get an ugly temporary label:

    00000000 <foo>:
       0:        81 70       andi       r24, 0x01    ; 1
       2:        80 30       cpi        r24, 0x00    ; 0
       4:        f9 f3       breq       .-2          ; 0x4 <foo+0x4>
       6:        83 e0       ldi        r24, 0x03    ; 3
       8:        08 95       ret

    0000000a <LBB0_2>:
       a:        85 e0       ldi        r24, 0x05    ; 5
       c:        08 95       ret

This patch fixes that, the output is now:

    00000000 <foo>:
       0:        81 70       andi       r24, 0x01    ; 1
       2:        80 30       cpi        r24, 0x00    ; 0
       4:        01 f0       breq       .+0          ; 0x6 <foo+0x6>
       6:        83 e0       ldi        r24, 0x03    ; 3
       8:        08 95       ret
       a:        85 e0       ldi        r24, 0x05    ; 5
       c:        08 95       ret

Note that as you can see the breq operand is different. However it is
still the same after linking:

       4:        11 f0       breq       .+4

Differential Revision: https://reviews.llvm.org/D75124
The file was modifiedllvm/lib/Target/AVR/MCTargetDesc/AVRMCAsmInfo.cpp (diff)
The file was modifiedllvm/test/CodeGen/AVR/rot.ll (diff)
The file was modifiedllvm/test/CodeGen/AVR/ctlz.ll (diff)
The file was modifiedllvm/test/CodeGen/AVR/branch-relaxation-long.ll (diff)
The file was modifiedllvm/test/CodeGen/AVR/umul-with-overflow.ll (diff)
The file was modifiedllvm/test/CodeGen/AVR/branch-relaxation.ll (diff)
The file was modifiedllvm/test/CodeGen/AVR/integration/blink.ll (diff)
The file was modifiedllvm/test/CodeGen/AVR/cttz.ll (diff)
The file was modifiedllvm/test/CodeGen/AVR/smul-with-overflow.ll (diff)
Commit 3e440545dc4404f5fdc3edceba96d901aeaef923 by nikita.ppv
[CVP] Add test for cmp of local phi; NFC
The file was modifiedllvm/test/Transforms/CorrelatedValuePropagation/icmp.ll (diff)
Commit 7da3b5e45c254c2366bd330fd4411f80011724ef by nikita.ppv
[InstCombine] Simplify DCE code; NFC

As pointed out on D75008, MadeIRChange is already set by
eraseInstFromFunction(), which also already does a debug print.
The file was modifiedllvm/lib/Transforms/InstCombine/InstructionCombining.cpp (diff)
Commit 3bf3996cd4ef2d4898b32d4cef52d6549bbe6820 by julian.lettner
[TSan] Add CMake check for minimal SDK requirements on Darwin

Fails with the following message in the error case:
```
CMake Error at /path/to/llvm-project/compiler-rt/lib/tsan/CMakeLists.txt:119 (message):
  Building the TSan runtime requires at least macOS SDK 10.12
```

Fixes #44682.
https://bugs.llvm.org/show_bug.cgi?id=44682

Reviewed By: dmajor, delcypher

Differential Revision: https://reviews.llvm.org/D74501
The file was modifiedcompiler-rt/lib/tsan/CMakeLists.txt (diff)
Commit 9d9633fb700d2c18b67b253a59145a543c6a37c4 by nikita.ppv
[CVP] Simplify cmp of local phi node

CVP currently does not simplify cmps with instructions in the same
block, because LVI getPredicateAt() currently does not provide
much useful information for that case (D69686 would change that,
but is stuck.) However, if the instruction is a Phi node, then
LVI can compute the result of the predicate by threading it into
the predecessor blocks, which allows it simplify some conditions
that nothing else can handle. Relevant code:
https://github.com/llvm/llvm-project/blob/6d6a4590c5d4c7fc7445d72fe685f966b0a8cafb/llvm/lib/Analysis/LazyValueInfo.cpp#L1904-L1927

Differential Revision: https://reviews.llvm.org/D72169
The file was modifiedllvm/test/Transforms/CorrelatedValuePropagation/icmp.ll (diff)
The file was modifiedllvm/lib/Transforms/Scalar/CorrelatedValuePropagation.cpp (diff)
Commit 00f54050f7e8d8ac0bab2a34affc98f6e4221dd4 by nikita.ppv
[SimpleLoopUnswitch] Remove unnecessary include; NFC
The file was modifiedllvm/lib/Transforms/Scalar/SimpleLoopUnswitch.cpp (diff)
Commit ca50f09db9f86eb5ac7aa0cb53c52637f2b7effa by tyker
[clang] fix error detection in consteval calls

Summary:
code like:
```
consteval int f() {
  int *A = new int(0);
  return *A;
}

int i1 = f();
```
currently doesn't generate any error.

Reviewers: rsmith

Reviewed By: rsmith

Differential Revision: https://reviews.llvm.org/D74418
The file was modifiedclang/lib/Sema/SemaExpr.cpp (diff)
The file was modifiedclang/test/SemaCXX/cxx2a-consteval.cpp (diff)
Commit 5bc6ff6455ec663a5da2681d057d0f848817b388 by antiagainst
[mlir][spirv] Add some folders for spv.LogicalAnd/spv.LogicalOr

This commit handles folding spv.LogicalAnd/spv.LogicalOr when
one of the operands is constant true/false.

Differential Revision: https://reviews.llvm.org/D75195
The file was modifiedmlir/lib/Dialect/SPIRV/SPIRVCanonicalization.cpp (diff)
The file was modifiedmlir/include/mlir/Dialect/SPIRV/SPIRVLogicalOps.td (diff)
The file was modifiedmlir/test/Dialect/SPIRV/canonicalize.mlir (diff)
Commit 25c6544f32eec4fd563f5531cec1d42b5c9c3e13 by spatel
[VectorCombine] add a debug flag to skip all transforms

As suggested in D75145 -

I'm not sure why, but several passes have this kind of disable/enable flag
implemented at the pass manager level. But that means we have to duplicate
the flag for both pass managers and add code to check the flag every time
the pass appears in the pipeline.

We want a debug option to see if this pass is misbehaving regardless of the
pass managers, so just add a disablement check at the single point before
any transforms run.

Differential Revision: https://reviews.llvm.org/D75204
The file was modifiedllvm/lib/Transforms/Vectorize/VectorCombine.cpp (diff)
Commit 85b641c27aecee637d3f7ab25915f47438f55848 by epastor
[ms] Rename ParsingInlineAsm functions/variables to reflect MS-specificity.

Summary: ParsingInlineAsm was a misleading name. These values are only set for MS-style inline assembly.

Reviewed By: rnk

Tags: #clang, #llvm

Differential Revision: https://reviews.llvm.org/D75198
The file was modifiedllvm/lib/MC/MCParser/MasmParser.cpp (diff)
The file was modifiedllvm/lib/Target/X86/AsmParser/X86AsmParser.cpp (diff)
The file was modifiedllvm/include/llvm/MC/MCParser/MCAsmParser.h (diff)
The file was modifiedllvm/include/llvm/MC/MCParser/MCTargetAsmParser.h (diff)
The file was modifiedclang/lib/Parse/ParseStmtAsm.cpp (diff)
The file was modifiedllvm/lib/MC/MCParser/AsmParser.cpp (diff)
Commit 22cb36c86b80fb9148fb08b1d384cd84bb9548d5 by Adrian Prantl
[dsymutil] Avoid copying swiftinterfaces from the SDK into the dsym bundle

This patch fixes a bug that would cause dsymutil to collect
.swiftinterface files for the Swift stdlib and other SDK
modules. There is no advantage in copying these since they should be
loaded from the ones bundled with LLDB's embedded Swift compiler
instead and copying them will cause LLDB to recompile them from source
instead of loading their prebuilt cached counterparts in the SDK.

rdar://problem/57463247

Differential Revisions: https://reviews.llvm.org/D75196
The file was modifiedllvm/test/tools/dsymutil/X86/swift-interface.test (diff)
The file was modifiedllvm/lib/DWARFLinker/DWARFLinker.cpp (diff)
The file was modifiedllvm/lib/DWARFLinker/DWARFLinkerCompileUnit.cpp (diff)
The file was removedllvm/test/tools/dsymutil/Inputs/swift-interface.ll
The file was addedllvm/test/tools/dsymutil/Inputs/swift-interface.s
The file was modifiedllvm/include/llvm/DWARFLinker/DWARFLinkerCompileUnit.h (diff)
Commit d2cbaf1755ffa90300365c0d71400a5ee4ada3bd by leonardchan
Fix for the test failues introduced in https://reviews.llvm.org/D74704.
The file was modifiedclang/test/Driver/riscv64-toolchain-extra.c (diff)
The file was modifiedclang/test/Driver/riscv32-toolchain-extra.c (diff)
Commit 52073310e443a206653f8f4c841f42ed88ab549b by paulatoth
[libc] Fix typo of namespace in round implementation.

Summary: Fixed typos in namespace declarations and comments in the files under `src/math/round`.

Reviewers: sivachandra, abrachet

Reviewed By: sivachandra, abrachet

Subscribers: MaskRay, tschuett, libc-commits

Tags: #libc-project

Differential Revision: https://reviews.llvm.org/D75205
The file was modifiedlibc/src/math/round/round_redirector.cpp (diff)
The file was modifiedlibc/src/math/round/round.cpp (diff)
The file was modifiedlibc/src/math/round/round.h (diff)
Commit c753a306fd1ad4a7e5c61367225abce86ac29cd7 by kostyak
[scudo][standalone] Various improvements wrt RSS

Summary:
This patch includes several changes to reduce the overall footprint
of the allocator:
- for realloc'd chunks: only keep the same chunk when lowering the size
  if the delta is within a page worth of bytes;
- when draining a cache: drain the beginning, not the end; we add pointers
  at the end, so that meant we were draining the most recently added
  pointers;
- change the release code to account for an freed up last page: when
  scanning the pages, we were looking for pages fully covered by blocks;
  in the event of the last page, if it's only partially covered, we
  wouldn't mark it as releasable - even what follows the last chunk is
  all 0s. So now mark the rest of the page as releasable, and adapt the
  test;
- add a missing `setReleaseToOsIntervalMs` to the cacheless secondary;
- adjust the Android classes based on more captures thanks to pcc@'s
  tool.

Reviewers: pcc, cferris, hctim, eugenis

Subscribers: #sanitizers, llvm-commits

Tags: #sanitizers, #llvm

Differential Revision: https://reviews.llvm.org/D75142
The file was modifiedcompiler-rt/lib/scudo/standalone/secondary.h (diff)
The file was modifiedcompiler-rt/lib/scudo/standalone/size_class_map.h (diff)
The file was modifiedcompiler-rt/lib/scudo/standalone/local_cache.h (diff)
The file was modifiedcompiler-rt/lib/scudo/standalone/tests/size_class_map_test.cpp (diff)
The file was modifiedcompiler-rt/lib/scudo/standalone/combined.h (diff)
The file was modifiedcompiler-rt/lib/scudo/standalone/tests/release_test.cpp (diff)
The file was modifiedcompiler-rt/lib/scudo/standalone/release.h (diff)
The file was modifiedcompiler-rt/lib/scudo/standalone/primary32.h (diff)
The file was modifiedcompiler-rt/lib/scudo/standalone/primary64.h (diff)
Commit 63779fb462d828d16b87f427a6490dded842ca15 by antiagainst
[mlir][spirv] Refactoring to avoid calling the same function twice
The file was modifiedmlir/lib/Dialect/SPIRV/SPIRVCanonicalization.cpp (diff)
Commit f79b81fbe2870ff631ad72a75f7e077b15b52a22 by alexshap
[llvm-objcopy] Enable --discard-all for MachO

In this diff we enable the option --discard-all for MachO.

Test plan: make check-all

Differential revision: https://reviews.llvm.org/D75104
The file was addedllvm/test/tools/llvm-objcopy/MachO/discard-all.test
The file was modifiedllvm/tools/llvm-objcopy/MachO/MachOObjcopy.cpp (diff)
Commit eca4bfea3d380f56652a19bfe4746663601050cb by listmail
[MC] Pull out a relaxFragment helper [NFC]

Having this as it's own function helps to reduce indentation and allows use of return instead of wiring a value over the switch.  A lambda would have also worked, but with slightly deeper nesting.
The file was modifiedllvm/include/llvm/MC/MCAssembler.h (diff)
The file was modifiedllvm/lib/MC/MCAssembler.cpp (diff)
Commit 7848a3c8ab54570f5875e63606ff75b5736750b4 by rjmccall
Update the ARC docs for non-trivial ownership in structs.
The file was modifiedclang/docs/AutomaticReferenceCounting.rst (diff)
Commit 949134e2fefd34a38ed71de90dffe2300e2e1139 by a.v.lapshin
[DebugInfo][NFC] Remove handler with ErrorPolicy from DWARFContext.

Summary:
Current LLVM code base does not use error handler with ErrorPolicy.
This patch removes ErrorPolicy from DWARFContext.

This patch is extracted from the D74308.

Reviewers: jhenderson, dblaikie, grimar, aprantl, JDevlieghere

Reviewed By: grimar

Subscribers: hiraditya, llvm-commits

Tags: #llvm, #debug-info

Differential Revision: https://reviews.llvm.org/D75118
The file was modifiedllvm/lib/DebugInfo/DWARF/DWARFContext.cpp (diff)
The file was modifiedllvm/unittests/DebugInfo/DWARF/DWARFDebugInfoTest.cpp (diff)
The file was modifiedllvm/include/llvm/DebugInfo/DWARF/DWARFContext.h (diff)
The file was modifiedllvm/lib/DebugInfo/Symbolize/Symbolize.cpp (diff)
Commit 16cabf278fc8c14d415e677ce0bc40d46a6de30d by n.james93
[ASTMatchers] HasNameMatcher handles `extern "C"`

Summary: Fixes [[ https://bugs.llvm.org/show_bug.cgi?id=42193 | hasName AST matcher is confused by extern "C" in namespace. ]]

Reviewers: klimek, aaron.ballman, gribozavr2

Reviewed By: aaron.ballman

Subscribers: cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D75202
The file was modifiedclang/unittests/ASTMatchers/ASTMatchersNarrowingTest.cpp (diff)
The file was modifiedclang/lib/ASTMatchers/ASTMatchersInternal.cpp (diff)
Commit 65f99b5383ff3293881f59dd64cfb596c3d03aa4 by Amara Emerson
[AArch64][GlobalISel] Fixup <32b heterogeneous regbanks of G_PHIs just before selection.

Since all types <32b on gpr end up being assigned gpr32 regclasses, we can end
up with PHIs here which try to select between a gpr32 and an fpr16. Ideally RBS
shouldn't be selecting heterogenous regbanks for operands if possible, but we
still need to be able to deal with it here.

To fix this, if we have a gpr-bank operand < 32b in size and at least one other
operand is on the fpr bank, then we add cross-bank copies to homogenize the
operand banks. For simplicity the bank that we choose to settle on is whatever
bank the def operand has. For example:

%endbb:
  %dst:gpr(s16) = G_PHI %in1:gpr(s16), %bb1, %in2:fpr(s16), %bb2
=>
%bb2:
  ...
  %in2_copy:gpr(s16) = COPY %in2:fpr(s16)
  ...
%endbb:
  %dst:gpr(s16) = G_PHI %in1:gpr(s16), %bb1, %in2_copy:gpr(s16), %bb2

Differential Revision: https://reviews.llvm.org/D75086
The file was addedllvm/test/CodeGen/AArch64/GlobalISel/preselect-process-phis.mir
The file was modifiedllvm/lib/Target/AArch64/AArch64InstructionSelector.cpp (diff)
Commit 34ee941f6d04454838456f0dc692f4abab5cdd19 by ditaliano
[ObjectFileMachO] Fix a build error on embedded.
The file was modifiedlldb/source/Plugins/ObjectFile/Mach-O/ObjectFileMachO.cpp (diff)
Commit 14306ce80c660fda31e2b5279b66174066326531 by craig.topper
[X86] Add proper MachinePointerInfo to the loads/stores created for moving data between SSE and X87 in X86DAGToDAGISel::PreprocessISelDAG
The file was modifiedllvm/lib/Target/X86/X86ISelDAGToDAG.cpp (diff)
Commit a8186935aea67e7a3041b83a50a83073a866e936 by craig.topper
[X86] Use correct MachineMemOperand for stack load in LowerFLT_ROUNDS_
The file was modifiedllvm/lib/Target/X86/X86ISelLowering.cpp (diff)
Commit 9c1a707ba3f6dbb684d0b50b8e301468e04c9d61 by craig.topper
[X86] Use proper alignment for stack temporary and correct MachinePointerInfo for stack accesses in LowerUINT_TO_FP.
The file was modifiedllvm/lib/Target/X86/X86ISelLowering.cpp (diff)
Commit 77d9b7b2cdbc401ded68970bc83efc6d1e179318 by craig.topper
[X86] Query constant pool object alignment instead of hardcoding.
The file was modifiedllvm/lib/Target/X86/X86ISelLowering.cpp (diff)
Commit a5a07b80419e68afd14916e95e2c613a9cee26d9 by Adrian Prantl
Visit previously unreachable nodes in the debug info metadata verifier.

This allows for diagnosing malformed LLVM IR debug info metadata such
as the one in the testcase.

<rdar://problem/59756060>

Differential Revision: https://reviews.llvm.org/D75212
The file was modifiedllvm/lib/IR/Verifier.cpp (diff)
The file was addedllvm/test/Verifier/llvm.loop.cu.ll
Commit 30f4362040fe28069e96c217856817e9014ab26d by erik.pilkington
Disable memory leak checking in a test to work around a bot failure

This leak is expected, see the discussion on commit 85fb997
The file was modifiedllvm/test/ExecutionEngine/OrcLazy/objc-minimal.ll (diff)
Commit 98ed0c5475df57ca5cd4df0997d8bba323c843aa by richard
PR44978: Accept as an extension some cases where destructor name lookup
is ambiguous, but only one of the possible lookup results could possibly
be right.

Clang recently started diagnosing ambiguity in more cases, and this
broke the build of Firefox. GCC, ICC, MSVC, and previous versions of
Clang all accept some forms of ambiguity here (albeit different ones in
each case); this patch mostly accepts anything any of those compilers
accept.
The file was modifiedclang/include/clang/Basic/DiagnosticSemaKinds.td (diff)
The file was modifiedclang/lib/Sema/SemaExprCXX.cpp (diff)
The file was modifiedclang/test/SemaCXX/destructor.cpp (diff)