FailedChanges

Summary

  1. X86: Replace isSafeToClobberEFLAGS implementation Also use modifiesRegister instead of looping over operands.
  2. Revert "Fix implementation of [temp.local]p4." This reverts commit 40bd10b770813bd1471d46f514545437516aa4ba. This seems to now emit an error when building the sanitizer tests: http://green.lab.llvm.org/green/job/clang-stage1-configure-RA/53965/consoleFull.
  3. Revert "[SystemZ] Do not emit VEXTEND or VROUND nodes without vector support." This reverts commit aa0b77d3395dc6ab91647138139c1a15a3aa088d. This fails to pass the machine verifier: http://green.lab.llvm.org/green/job/clang-stage1-cmake-RA-expensive/13579/
  4. [lit][NFC] Cleanup copy&paste naming mistake
  5. [llvm-cxxfilt] Fix a comment typo. NFC.
  6. [GISel][NFC]: Add methods to speed up insertion into GISelWorklist https://reviews.llvm.org/D58073 Speed up insertion during the initial populating phase into the GISelWorkList by deferring repeatedly resizing the DenseMap. This results in ~10% improvement in the combiner passes, and ~3% speedup in the Legalizer. reviewed by: aemerson.
  7. [libFuzzer] print new functions as they are discovered in the fork mode
  8. Fix implementation of [temp.local]p4. When a template-name is looked up, we need to give injected-class-name declarations of class templates special treatment, as they denote a template rather than a type. Previously we achieved this by applying a filter to the lookup results after completing name lookup, but that is incorrect in various ways, not least of which is that it lost all information about access and how members were named, and the filtering caused us to generally lose all ambiguity errors between templates and non-templates. We now preserve the lookup results exactly, and the few places that need to map from a declaration found by name lookup into a declaration of a template do so explicitly. Deduplication of repeated lookup results of the same injected-class-name declaration is done by name lookup instead of after the fact.
  9. PR40642: Fix determination of whether the final statement of a statement expression is a discarded-value expression. Summary: We used to get this wrong in three ways: 1) During parsing, an expression-statement followed by the }) ending a statement expression was always treated as producing the value of the statement expression. That's wrong for ({ if (1) expr; }) 2) During template instantiation, various kinds of statement (most statements not appearing directly in a compound-statement) were not treated as discarded-value expressions, resulting in missing volatile loads (etc). 3) In all contexts, an expression-statement with attributes was not treated as producing the value of the statement expression, eg ({ [[attr]] expr; }). Also fix incorrect enforcement of OpenMP rule that directives can "only be placed in the program at a position where ignoring or deleting the directive would result in a program with correct syntax". In particular, a label (be it goto, case, or default) should not affect whether directives are permitted. Reviewers: aaron.ballman, rjmccall Subscribers: cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D57984
  10. Fixed failure on Darwin due to r354064 Summary: instrprof-darwin-exports.c test fails on Darwin due to r354064. Updated clang list of exported symbols to fix the issue. Reviewers: vsk Reviewed By: vsk Subscribers: davidxl, efriedma Differential Revision: https://reviews.llvm.org/D58259
  11. [libFuzzer] fix the unit tests
  12. [libFuzzer] when doing the merge, keep track of the coveraged edges, not just features
  13. AMDGPU: Set ABI version to 1 for code object v3 Differential Revision: https://reviews.llvm.org/D57811
  14. [Driver][Darwin] Emit an error when using -pg on OS without support for it. Instead of letting a program fail at runtime, emit an error during compilation. rdar://problem/12206955 Reviewers: dexonsmith, bob.wilson, steven_wu Reviewed By: steven_wu Subscribers: jkorous, cfe-commits Differential Revision: https://reviews.llvm.org/D57991
  15. [symbolizer] Avoid collecting symbols belonging to invalid sections. Summary: llvm-symbolizer would originally report symbols that belonged to an invalid object file section. Specifically the case where: `*Symbol.getSection() == ObjFile.section_end()` This patch prevents the Symbolizer from collecting symbols that belong to invalid sections. The test (from PR40591) introduces a case where two symbols have address 0, one symbol is defined, 'foo', and the other is not defined, 'bar'. This patch will cause the Symbolizer to keep 'foo' and ignore 'bar'. As a side note, the logic for adding symbols to the Symbolizer's store (`SymbolizableObjectFile::addSymbol`) replaces symbols with the same <address, size> pair. At some point that logic should be revisited as in the aforementioned case, 'bar' was overwriting 'foo' in the Symbolizer's store, and 'foo' was forgotten. This fixes PR40591 Reviewers: jhenderson, rupprecht Reviewed By: rupprecht Subscribers: hiraditya, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D58146
  16. Revert "[INLINER] allow inlining of address taken blocks" This reverts commit 19e95fe61182945b7b68ad15348f144fb996633f.
  17. [INLINER] allow inlining of address taken blocks as long as their uses does not contain calls to functions that capture the argument (potentially allowing the blockaddress to "escape" the lifetime of the caller). TODO: - add more tests - fix crash in llvm::updateCGAndAnalysisManagerForFunctionPass when invoking Transforms/Inline/blockaddress.ll
  18. [x86] add tests for extractelement of FP; NFC
  19. [libFuzzer] when doing the merge, keep track of the coveraged edges, not just features
  20. [clang][FileManager] fillRealPathName even if we aren't opening the file The pathname wasn't previously filled when the getFile() method was called with openFile = false. We are caching FileEntry-s in ParsedAST::Includes in clangd and this caused the problem. This fixes an internal test failure in clangd - ClangdTests.GoToInclude.All rdar://47536127 Differential Revision: https://reviews.llvm.org/D58213
  21. [Sema] Fix-up a -Wfloat-conversion diagnostic We were warning on valid ObjC property reference exprs, and passing in the wrong arguments to DiagnoseFloatingImpCast (leading to a badly worded diagnostic). rdar://47644670 Differential revision: https://reviews.llvm.org/D58145
  22. [lit] Remove --single-process option (use -j1 instead) Remove `--single-process` command line option. Use `-j1` instead. Also see commit: 96adb78b120b6aa9739eb714534dc8e819f7bc52
  23. MC/ELF: Allow targets to set ABI version Tests are in the follow up change Differential Revision: https://reviews.llvm.org/D57810
  24. GlobalISel: Add alignment to LegalityQuery MMOs This allows targets to specify the minimum alignment required for the load/store.
  25. Replace gcroot verifier tests These haven't been checking anything useful and have been testing the wrong failure reason for many years. Replace them with something which stresses what is actually implemented in the verifier now.
  26. [clang-tidy] Mention language version in test explicitly. "modernize-use-using" check is applicable only to C++11 and later. Spell it out to avoid relying on default language version. rdar://problem/47932196
  27. [lit] Set --single-process for single tests and --threads=1 Summary: Automatically upgrade debugging experience (single process, no thread pool) when: 1) we only run a single test 2) user specifies `-j1` Details: Fix `--max-failures` in single process mode. Option did not have an effect in single process mode. Add display feedback for single process mode. Adapted test. Improve argument checking (require positive integers). `--single-process` is now essentially an alias for `-j1`. Should we remove it? Reviewers: rnk Differential Revision: https://reviews.llvm.org/D58249
  28. AMDGPU/GlobalISel: Fix RegBankSelect for GEP. This is basically a pointer typed add, so shouldn't be any different. This was assuming everything was an SGPR, which is not true. Also cleanup legality for GEP. I don't seem to be seeing the problem the hack marking s64 as a legal pointer type the comment mentions.
  29. [AMDGPU] Ressociate 'add (add x, y), z' to use SALU Reassociate adds to collect scalar operands in a single instruction when possible. That will result in a scalar add followed by vector instead of two vector adds, thus better utilizing SALU. Differential Revision: https://reviews.llvm.org/D58220
  30. AMDGPU/GlobalISel: Handle split for 64-bit VALU select
Revision 354098 by arsenm:
X86: Replace isSafeToClobberEFLAGS implementation

Also use modifiesRegister instead of looping over operands.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/X86/X86InstrInfo.cpp (diff)llvm.src/lib/Target/X86/X86InstrInfo.cpp
The file was modified/llvm/trunk/lib/Target/X86/X86InstrInfo.h (diff)llvm.src/lib/Target/X86/X86InstrInfo.h
Revision 354097 by thegameg:
Revert "Fix implementation of [temp.local]p4."

This reverts commit 40bd10b770813bd1471d46f514545437516aa4ba.

This seems to now emit an error when building the sanitizer tests:
http://green.lab.llvm.org/green/job/clang-stage1-configure-RA/53965/consoleFull.
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/include/clang/Sema/Lookup.h (diff)clang.src/include/clang/Sema/Lookup.h
The file was modified/cfe/trunk/include/clang/Sema/Sema.h (diff)clang.src/include/clang/Sema/Sema.h
The file was modified/cfe/trunk/lib/Sema/SemaDecl.cpp (diff)clang.src/lib/Sema/SemaDecl.cpp
The file was modified/cfe/trunk/lib/Sema/SemaDeclCXX.cpp (diff)clang.src/lib/Sema/SemaDeclCXX.cpp
The file was modified/cfe/trunk/lib/Sema/SemaLookup.cpp (diff)clang.src/lib/Sema/SemaLookup.cpp
The file was modified/cfe/trunk/lib/Sema/SemaTemplate.cpp (diff)clang.src/lib/Sema/SemaTemplate.cpp
The file was modified/cfe/trunk/test/CXX/class.access/p4.cpp (diff)clang.src/test/CXX/class.access/p4.cpp
The file was modified/cfe/trunk/test/CXX/temp/temp.decls/temp.friend/p1.cpp (diff)clang.src/test/CXX/temp/temp.decls/temp.friend/p1.cpp
The file was modified/cfe/trunk/test/SemaTemplate/temp.cpp (diff)clang.src/test/SemaTemplate/temp.cpp
Revision 354096 by thegameg:
Revert "[SystemZ] Do not emit VEXTEND or VROUND nodes without vector support."

This reverts commit aa0b77d3395dc6ab91647138139c1a15a3aa088d.

This fails to pass the machine verifier:
http://green.lab.llvm.org/green/job/clang-stage1-cmake-RA-expensive/13579/
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/SystemZ/SystemZISelLowering.cpp (diff)llvm.src/lib/Target/SystemZ/SystemZISelLowering.cpp
The file was removed/llvm/trunk/test/CodeGen/SystemZ/fp-conv-18.mirllvm.src/test/CodeGen/SystemZ/fp-conv-18.mir
Revision 354095 by yln:
[lit][NFC] Cleanup copy&paste naming mistake
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/utils/lit/tests/Inputs/progress-bar/lit.cfg (diff)llvm.src/utils/lit/tests/Inputs/progress-bar/lit.cfg
The file was modified/llvm/trunk/utils/lit/tests/progress-bar.py (diff)llvm.src/utils/lit/tests/progress-bar.py
Revision 354094 by mattd:
[llvm-cxxfilt] Fix a comment typo. NFC.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/tools/llvm-cxxfilt/llvm-cxxfilt.cpp (diff)llvm.src/tools/llvm-cxxfilt/llvm-cxxfilt.cpp
Revision 354093 by aditya_nandakumar:
[GISel][NFC]: Add methods to speed up insertion into GISelWorklist

https://reviews.llvm.org/D58073

Speed up insertion during the initial populating phase into the
GISelWorkList by deferring repeatedly resizing the DenseMap.
This results in ~10% improvement in the combiner passes, and
~3% speedup in the Legalizer.

reviewed by: aemerson.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/CodeGen/GlobalISel/GISelWorkList.h (diff)llvm.src/include/llvm/CodeGen/GlobalISel/GISelWorkList.h
The file was modified/llvm/trunk/lib/CodeGen/GlobalISel/Combiner.cpp (diff)llvm.src/lib/CodeGen/GlobalISel/Combiner.cpp
The file was modified/llvm/trunk/lib/CodeGen/GlobalISel/Legalizer.cpp (diff)llvm.src/lib/CodeGen/GlobalISel/Legalizer.cpp
Revision 354092 by kcc:
[libFuzzer] print new functions as they are discovered in the fork mode
Change TypePath in RepositoryPath in Workspace
The file was modified/compiler-rt/trunk/lib/fuzzer/FuzzerFork.cpp (diff)compiler-rt.src/lib/fuzzer/FuzzerFork.cpp
The file was modified/compiler-rt/trunk/lib/fuzzer/FuzzerMerge.cpp (diff)compiler-rt.src/lib/fuzzer/FuzzerMerge.cpp
The file was modified/compiler-rt/trunk/lib/fuzzer/FuzzerTracePC.cpp (diff)compiler-rt.src/lib/fuzzer/FuzzerTracePC.cpp
The file was modified/compiler-rt/trunk/lib/fuzzer/FuzzerTracePC.h (diff)compiler-rt.src/lib/fuzzer/FuzzerTracePC.h
Revision 354091 by rsmith:
Fix implementation of [temp.local]p4.

When a template-name is looked up, we need to give injected-class-name
declarations of class templates special treatment, as they denote a
template rather than a type.

Previously we achieved this by applying a filter to the lookup results
after completing name lookup, but that is incorrect in various ways, not
least of which is that it lost all information about access and how
members were named, and the filtering caused us to generally lose
all ambiguity errors between templates and non-templates.

We now preserve the lookup results exactly, and the few places that need
to map from a declaration found by name lookup into a declaration of a
template do so explicitly. Deduplication of repeated lookup results of
the same injected-class-name declaration is done by name lookup instead
of after the fact.
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/include/clang/Sema/Lookup.h (diff)clang.src/include/clang/Sema/Lookup.h
The file was modified/cfe/trunk/include/clang/Sema/Sema.h (diff)clang.src/include/clang/Sema/Sema.h
The file was modified/cfe/trunk/lib/Sema/SemaDecl.cpp (diff)clang.src/lib/Sema/SemaDecl.cpp
The file was modified/cfe/trunk/lib/Sema/SemaDeclCXX.cpp (diff)clang.src/lib/Sema/SemaDeclCXX.cpp
The file was modified/cfe/trunk/lib/Sema/SemaLookup.cpp (diff)clang.src/lib/Sema/SemaLookup.cpp
The file was modified/cfe/trunk/lib/Sema/SemaTemplate.cpp (diff)clang.src/lib/Sema/SemaTemplate.cpp
The file was modified/cfe/trunk/test/CXX/class.access/p4.cpp (diff)clang.src/test/CXX/class.access/p4.cpp
The file was modified/cfe/trunk/test/CXX/temp/temp.decls/temp.friend/p1.cpp (diff)clang.src/test/CXX/temp/temp.decls/temp.friend/p1.cpp
The file was modified/cfe/trunk/test/SemaTemplate/temp.cpp (diff)clang.src/test/SemaTemplate/temp.cpp
Revision 354090 by rsmith:
PR40642: Fix determination of whether the final statement of a statement
expression is a discarded-value expression.

Summary:
We used to get this wrong in three ways:

1) During parsing, an expression-statement followed by the }) ending a
   statement expression was always treated as producing the value of the
   statement expression. That's wrong for ({ if (1) expr; })
2) During template instantiation, various kinds of statement (most
   statements not appearing directly in a compound-statement) were not
   treated as discarded-value expressions, resulting in missing volatile
   loads (etc).
3) In all contexts, an expression-statement with attributes was not
   treated as producing the value of the statement expression, eg
   ({ [[attr]] expr; }).

Also fix incorrect enforcement of OpenMP rule that directives can "only
be placed in the program at a position where ignoring or deleting the
directive would result in a program with correct syntax". In particular,
a label (be it goto, case, or default) should not affect whether
directives are permitted.

Reviewers: aaron.ballman, rjmccall

Subscribers: cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D57984
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/include/clang/AST/Expr.h (diff)clang.src/include/clang/AST/Expr.h
The file was modified/cfe/trunk/include/clang/AST/Stmt.h (diff)clang.src/include/clang/AST/Stmt.h
The file was modified/cfe/trunk/include/clang/Basic/StmtNodes.td (diff)clang.src/include/clang/Basic/StmtNodes.td
The file was modified/cfe/trunk/include/clang/Parse/Parser.h (diff)clang.src/include/clang/Parse/Parser.h
The file was modified/cfe/trunk/include/clang/Sema/Sema.h (diff)clang.src/include/clang/Sema/Sema.h
The file was modified/cfe/trunk/lib/AST/Stmt.cpp (diff)clang.src/lib/AST/Stmt.cpp
The file was modified/cfe/trunk/lib/CodeGen/CGStmt.cpp (diff)clang.src/lib/CodeGen/CGStmt.cpp
The file was modified/cfe/trunk/lib/Parse/ParseObjc.cpp (diff)clang.src/lib/Parse/ParseObjc.cpp
The file was modified/cfe/trunk/lib/Parse/ParseOpenMP.cpp (diff)clang.src/lib/Parse/ParseOpenMP.cpp
The file was modified/cfe/trunk/lib/Parse/ParseStmt.cpp (diff)clang.src/lib/Parse/ParseStmt.cpp
The file was modified/cfe/trunk/lib/Sema/SemaExpr.cpp (diff)clang.src/lib/Sema/SemaExpr.cpp
The file was modified/cfe/trunk/lib/Sema/SemaStmt.cpp (diff)clang.src/lib/Sema/SemaStmt.cpp
The file was modified/cfe/trunk/lib/Sema/TreeTransform.h (diff)clang.src/lib/Sema/TreeTransform.h
The file was modified/cfe/trunk/test/CodeGenCXX/stmtexpr.cpp (diff)clang.src/test/CodeGenCXX/stmtexpr.cpp
The file was modified/cfe/trunk/test/CodeGenCXX/volatile.cpp (diff)clang.src/test/CodeGenCXX/volatile.cpp
The file was modified/cfe/trunk/test/OpenMP/barrier_messages.cpp (diff)clang.src/test/OpenMP/barrier_messages.cpp
The file was modified/cfe/trunk/test/OpenMP/cancel_messages.cpp (diff)clang.src/test/OpenMP/cancel_messages.cpp
The file was modified/cfe/trunk/test/OpenMP/cancellation_point_messages.cpp (diff)clang.src/test/OpenMP/cancellation_point_messages.cpp
The file was modified/cfe/trunk/test/OpenMP/flush_messages.cpp (diff)clang.src/test/OpenMP/flush_messages.cpp
The file was modified/cfe/trunk/test/OpenMP/taskwait_messages.cpp (diff)clang.src/test/OpenMP/taskwait_messages.cpp
The file was modified/cfe/trunk/test/OpenMP/taskyield_messages.cpp (diff)clang.src/test/OpenMP/taskyield_messages.cpp
Revision 354089 by apazos:
Fixed failure on Darwin due to r354064

Summary:
instrprof-darwin-exports.c test fails on Darwin due  to r354064.

Updated clang list of exported symbols to fix the issue.

Reviewers: vsk

Reviewed By: vsk

Subscribers: davidxl, efriedma

Differential Revision: https://reviews.llvm.org/D58259
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/lib/Driver/ToolChains/Darwin.cpp (diff)clang.src/lib/Driver/ToolChains/Darwin.cpp
Revision 354088 by kcc:
[libFuzzer] fix the unit tests
Change TypePath in RepositoryPath in Workspace
The file was modified/compiler-rt/trunk/lib/fuzzer/FuzzerMerge.cpp (diff)compiler-rt.src/lib/fuzzer/FuzzerMerge.cpp
The file was modified/compiler-rt/trunk/lib/fuzzer/FuzzerMerge.h (diff)compiler-rt.src/lib/fuzzer/FuzzerMerge.h
The file was modified/compiler-rt/trunk/lib/fuzzer/tests/FuzzerUnittest.cpp (diff)compiler-rt.src/lib/fuzzer/tests/FuzzerUnittest.cpp
Revision 354087 by kcc:
[libFuzzer] when doing the merge, keep track of the coveraged edges, not just features
Change TypePath in RepositoryPath in Workspace
The file was modified/compiler-rt/trunk/lib/fuzzer/FuzzerDriver.cpp (diff)compiler-rt.src/lib/fuzzer/FuzzerDriver.cpp
The file was modified/compiler-rt/trunk/lib/fuzzer/FuzzerFork.cpp (diff)compiler-rt.src/lib/fuzzer/FuzzerFork.cpp
The file was modified/compiler-rt/trunk/lib/fuzzer/FuzzerMerge.cpp (diff)compiler-rt.src/lib/fuzzer/FuzzerMerge.cpp
The file was modified/compiler-rt/trunk/lib/fuzzer/FuzzerMerge.h (diff)compiler-rt.src/lib/fuzzer/FuzzerMerge.h
Revision 354085 by kzhuravl:
AMDGPU: Set ABI version to 1 for code object v3

Differential Revision: https://reviews.llvm.org/D57811
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/AMDGPU/MCTargetDesc/AMDGPUAsmBackend.cpp (diff)llvm.src/lib/Target/AMDGPU/MCTargetDesc/AMDGPUAsmBackend.cpp
The file was modified/llvm/trunk/lib/Target/AMDGPU/MCTargetDesc/AMDGPUELFObjectWriter.cpp (diff)llvm.src/lib/Target/AMDGPU/MCTargetDesc/AMDGPUELFObjectWriter.cpp
The file was modified/llvm/trunk/lib/Target/AMDGPU/MCTargetDesc/AMDGPUMCTargetDesc.h (diff)llvm.src/lib/Target/AMDGPU/MCTargetDesc/AMDGPUMCTargetDesc.h
The file was modified/llvm/trunk/test/CodeGen/AMDGPU/elf-header-osabi.ll (diff)llvm.src/test/CodeGen/AMDGPU/elf-header-osabi.ll
Revision 354084 by vsapsai:
[Driver][Darwin] Emit an error when using -pg on OS without support for it.

Instead of letting a program fail at runtime, emit an error during
compilation.

rdar://problem/12206955

Reviewers: dexonsmith, bob.wilson, steven_wu

Reviewed By: steven_wu

Subscribers: jkorous, cfe-commits

Differential Revision: https://reviews.llvm.org/D57991
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/include/clang/Basic/DiagnosticDriverKinds.td (diff)clang.src/include/clang/Basic/DiagnosticDriverKinds.td
The file was modified/cfe/trunk/lib/Driver/ToolChains/Darwin.cpp (diff)clang.src/lib/Driver/ToolChains/Darwin.cpp
The file was modified/cfe/trunk/test/Driver/darwin-ld.c (diff)clang.src/test/Driver/darwin-ld.c
Revision 354083 by mattd:
[symbolizer] Avoid collecting symbols belonging to invalid sections.

Summary:
llvm-symbolizer would originally report symbols that belonged to an invalid object file section.
Specifically the case where: `*Symbol.getSection() == ObjFile.section_end()`
This patch prevents the Symbolizer from collecting symbols that belong to invalid sections.

The test  (from PR40591) introduces a case where two symbols have address 0,
one symbol is defined, 'foo', and the other is not defined, 'bar'.  This patch will cause
the Symbolizer to keep 'foo' and ignore 'bar'.

As a side note, the logic for adding symbols to the Symbolizer's store
(`SymbolizableObjectFile::addSymbol`) replaces symbols with the
same <address, size> pair.  At some point that logic should be revisited as in the
aforementioned case, 'bar' was overwriting 'foo' in the Symbolizer's store,
and 'foo' was forgotten.

This fixes PR40591

Reviewers: jhenderson, rupprecht

Reviewed By: rupprecht

Subscribers: hiraditya, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D58146
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/DebugInfo/Symbolize/SymbolizableObjectFile.cpp (diff)llvm.src/lib/DebugInfo/Symbolize/SymbolizableObjectFile.cpp
The file was added/llvm/trunk/test/tools/llvm-symbolizer/ignore-undefined-symbols.sllvm.src/test/tools/llvm-symbolizer/ignore-undefined-symbols.s
Revision 354082 by nickdesaulniers:
Revert "[INLINER] allow inlining of address taken blocks"

This reverts commit 19e95fe61182945b7b68ad15348f144fb996633f.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/IR/BasicBlock.h (diff)llvm.src/include/llvm/IR/BasicBlock.h
The file was modified/llvm/trunk/lib/Analysis/InlineCost.cpp (diff)llvm.src/lib/Analysis/InlineCost.cpp
The file was modified/llvm/trunk/lib/IR/BasicBlock.cpp (diff)llvm.src/lib/IR/BasicBlock.cpp
Revision 354079 by nickdesaulniers:
[INLINER] allow inlining of address taken blocks

as long as their uses does not contain calls to functions that capture
the argument (potentially allowing the blockaddress to "escape" the
lifetime of the caller).

TODO:
- add more tests
- fix crash in llvm::updateCGAndAnalysisManagerForFunctionPass when
  invoking Transforms/Inline/blockaddress.ll
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/IR/BasicBlock.h (diff)llvm.src/include/llvm/IR/BasicBlock.h
The file was modified/llvm/trunk/lib/Analysis/InlineCost.cpp (diff)llvm.src/lib/Analysis/InlineCost.cpp
The file was modified/llvm/trunk/lib/IR/BasicBlock.cpp (diff)llvm.src/lib/IR/BasicBlock.cpp
Revision 354077 by spatel:
[x86] add tests for extractelement of FP; NFC
Change TypePath in RepositoryPath in Workspace
The file was added/llvm/trunk/test/CodeGen/X86/extractelement-fp.llllvm.src/test/CodeGen/X86/extractelement-fp.ll
Revision 354076 by kcc:
[libFuzzer] when doing the merge, keep track of the coveraged edges, not just features
Change TypePath in RepositoryPath in Workspace
The file was modified/compiler-rt/trunk/lib/fuzzer/FuzzerLoop.cpp (diff)compiler-rt.src/lib/fuzzer/FuzzerLoop.cpp
The file was modified/compiler-rt/trunk/lib/fuzzer/FuzzerMerge.cpp (diff)compiler-rt.src/lib/fuzzer/FuzzerMerge.cpp
The file was modified/compiler-rt/trunk/lib/fuzzer/FuzzerMerge.h (diff)compiler-rt.src/lib/fuzzer/FuzzerMerge.h
The file was modified/compiler-rt/trunk/lib/fuzzer/FuzzerTracePC.cpp (diff)compiler-rt.src/lib/fuzzer/FuzzerTracePC.cpp
The file was modified/compiler-rt/trunk/lib/fuzzer/FuzzerTracePC.h (diff)compiler-rt.src/lib/fuzzer/FuzzerTracePC.h
The file was modified/compiler-rt/trunk/lib/fuzzer/tests/FuzzerUnittest.cpp (diff)compiler-rt.src/lib/fuzzer/tests/FuzzerUnittest.cpp
The file was modified/compiler-rt/trunk/test/fuzzer/merge-control-file.test (diff)compiler-rt.src/test/fuzzer/merge-control-file.test
The file was modified/compiler-rt/trunk/test/fuzzer/merge-sigusr.test (diff)compiler-rt.src/test/fuzzer/merge-sigusr.test
The file was modified/compiler-rt/trunk/test/fuzzer/merge.test (diff)compiler-rt.src/test/fuzzer/merge.test
Revision 354075 by Jan Korous:
[clang][FileManager] fillRealPathName even if we aren't opening the file

The pathname wasn't previously filled when the getFile() method was called with openFile = false.
We are caching FileEntry-s in ParsedAST::Includes in clangd and this caused the problem.

This fixes an internal test failure in clangd - ClangdTests.GoToInclude.All

rdar://47536127

Differential Revision: https://reviews.llvm.org/D58213
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/lib/Basic/FileManager.cpp (diff)clang.src/lib/Basic/FileManager.cpp
The file was modified/cfe/trunk/unittests/Basic/FileManagerTest.cpp (diff)clang.src/unittests/Basic/FileManagerTest.cpp
Revision 354074 by epilk:
[Sema] Fix-up a -Wfloat-conversion diagnostic

We were warning on valid ObjC property reference exprs, and passing
in the wrong arguments to DiagnoseFloatingImpCast (leading to a badly
worded diagnostic).

rdar://47644670

Differential revision: https://reviews.llvm.org/D58145
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/lib/Sema/SemaChecking.cpp (diff)clang.src/lib/Sema/SemaChecking.cpp
The file was modified/cfe/trunk/test/SemaCXX/warn-float-conversion.cpp (diff)clang.src/test/SemaCXX/warn-float-conversion.cpp
The file was modified/cfe/trunk/test/SemaObjC/conversion.m (diff)clang.src/test/SemaObjC/conversion.m
Revision 354073 by yln:
[lit] Remove --single-process option (use -j1 instead)

Remove `--single-process` command line option. Use `-j1` instead.

Also see commit: 96adb78b120b6aa9739eb714534dc8e819f7bc52
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/utils/lit/lit/main.py (diff)llvm.src/utils/lit/lit/main.py
The file was modified/llvm/trunk/utils/lit/tests/discovery.py (diff)llvm.src/utils/lit/tests/discovery.py
Revision 354072 by kzhuravl:
MC/ELF: Allow targets to set ABI version

Tests are in the follow up change

Differential Revision: https://reviews.llvm.org/D57810
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/MC/MCELFObjectWriter.h (diff)llvm.src/include/llvm/MC/MCELFObjectWriter.h
The file was modified/llvm/trunk/lib/MC/ELFObjectWriter.cpp (diff)llvm.src/lib/MC/ELFObjectWriter.cpp
The file was modified/llvm/trunk/lib/MC/MCELFObjectTargetWriter.cpp (diff)llvm.src/lib/MC/MCELFObjectTargetWriter.cpp
Revision 354071 by arsenm:
GlobalISel: Add alignment to LegalityQuery MMOs

This allows targets to specify the minimum alignment required for the
load/store.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/CodeGen/GlobalISel/LegalizerInfo.h (diff)llvm.src/include/llvm/CodeGen/GlobalISel/LegalizerInfo.h
The file was modified/llvm/trunk/lib/CodeGen/GlobalISel/LegalityPredicates.cpp (diff)llvm.src/lib/CodeGen/GlobalISel/LegalityPredicates.cpp
The file was modified/llvm/trunk/lib/CodeGen/GlobalISel/LegalizerInfo.cpp (diff)llvm.src/lib/CodeGen/GlobalISel/LegalizerInfo.cpp
The file was modified/llvm/trunk/lib/Target/AArch64/AArch64LegalizerInfo.cpp (diff)llvm.src/lib/Target/AArch64/AArch64LegalizerInfo.cpp
The file was modified/llvm/trunk/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp (diff)llvm.src/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp
The file was modified/llvm/trunk/lib/Target/ARM/ARMLegalizerInfo.cpp (diff)llvm.src/lib/Target/ARM/ARMLegalizerInfo.cpp
The file was modified/llvm/trunk/lib/Target/Mips/MipsLegalizerInfo.cpp (diff)llvm.src/lib/Target/Mips/MipsLegalizerInfo.cpp
The file was modified/llvm/trunk/unittests/CodeGen/GlobalISel/LegalizerInfoTest.cpp (diff)llvm.src/unittests/CodeGen/GlobalISel/LegalizerInfoTest.cpp
Revision 354070 by arsenm:
Replace gcroot verifier tests

These haven't been checking anything useful and have been testing the
wrong failure reason for many years. Replace them with something which
stresses what is actually implemented in the verifier now.
Change TypePath in RepositoryPath in Workspace
The file was removed/llvm/trunk/test/Verifier/gcroot-alloca.llllvm.src/test/Verifier/gcroot-alloca.ll
The file was removed/llvm/trunk/test/Verifier/gcroot-meta.llllvm.src/test/Verifier/gcroot-meta.ll
The file was removed/llvm/trunk/test/Verifier/gcroot-ptrptr.llllvm.src/test/Verifier/gcroot-ptrptr.ll
The file was added/llvm/trunk/test/Verifier/gcroot.llllvm.src/test/Verifier/gcroot.ll
Revision 354069 by vsapsai:
[clang-tidy] Mention language version in test explicitly.

"modernize-use-using" check is applicable only to C++11 and later. Spell it out
to avoid relying on default language version.

rdar://problem/47932196
Change TypePath in RepositoryPath in Workspace
The file was modified/clang-tools-extra/trunk/test/clang-tidy/clang-tidy-mac-libcxx.cpp (diff)clang-tools-extra.src/test/clang-tidy/clang-tidy-mac-libcxx.cpp
Revision 354068 by yln:
[lit] Set --single-process for single tests and --threads=1

Summary:
Automatically upgrade debugging experience (single process, no thread
pool) when:
  1) we only run a single test
  2) user specifies `-j1`

Details:
Fix `--max-failures` in single process mode. Option did not have an
effect in single process mode.

Add display feedback for single process mode. Adapted test.

Improve argument checking (require positive integers).

`--single-process` is now essentially an alias for `-j1`. Should we
remove it?

Reviewers: rnk

Differential Revision: https://reviews.llvm.org/D58249
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/utils/lit/lit/LitConfig.py (diff)llvm.src/utils/lit/lit/LitConfig.py
The file was modified/llvm/trunk/utils/lit/lit/discovery.py (diff)llvm.src/utils/lit/lit/discovery.py
The file was modified/llvm/trunk/utils/lit/lit/main.py (diff)llvm.src/utils/lit/lit/main.py
The file was modified/llvm/trunk/utils/lit/lit/run.py (diff)llvm.src/utils/lit/lit/run.py
The file was modified/llvm/trunk/utils/lit/tests/max-failures.py (diff)llvm.src/utils/lit/tests/max-failures.py
The file was modified/llvm/trunk/utils/lit/tests/unit/TestRunner.py (diff)llvm.src/utils/lit/tests/unit/TestRunner.py
Revision 354067 by arsenm:
AMDGPU/GlobalISel: Fix RegBankSelect for GEP.

This is basically a pointer typed add, so shouldn't be any different.
This was assuming everything was an SGPR, which is not true.

Also cleanup legality for GEP. I don't seem to be seeing the problem
the hack marking s64 as a legal pointer type the comment mentions.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp (diff)llvm.src/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp
The file was modified/llvm/trunk/lib/Target/AMDGPU/AMDGPURegisterBankInfo.cpp (diff)llvm.src/lib/Target/AMDGPU/AMDGPURegisterBankInfo.cpp
The file was added/llvm/trunk/test/CodeGen/AMDGPU/GlobalISel/regbankselect-gep.mirllvm.src/test/CodeGen/AMDGPU/GlobalISel/regbankselect-gep.mir
Revision 354066 by rampitec:
[AMDGPU] Ressociate 'add (add x, y), z' to use SALU

Reassociate adds to collect scalar operands in a single
instruction when possible. That will result in a scalar
add followed by vector instead of two vector adds, thus
better utilizing SALU.

Differential Revision: https://reviews.llvm.org/D58220
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/AMDGPU/SIISelLowering.cpp (diff)llvm.src/lib/Target/AMDGPU/SIISelLowering.cpp
The file was modified/llvm/trunk/lib/Target/AMDGPU/SIISelLowering.h (diff)llvm.src/lib/Target/AMDGPU/SIISelLowering.h
The file was modified/llvm/trunk/test/CodeGen/AMDGPU/add3.ll (diff)llvm.src/test/CodeGen/AMDGPU/add3.ll
The file was added/llvm/trunk/test/CodeGen/AMDGPU/reassoc-scalar.llllvm.src/test/CodeGen/AMDGPU/reassoc-scalar.ll
Revision 354065 by arsenm:
AMDGPU/GlobalISel: Handle split for 64-bit VALU select
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/AMDGPU/AMDGPUGenRegisterBankInfo.def (diff)llvm.src/lib/Target/AMDGPU/AMDGPUGenRegisterBankInfo.def
The file was modified/llvm/trunk/lib/Target/AMDGPU/AMDGPURegisterBankInfo.cpp (diff)llvm.src/lib/Target/AMDGPU/AMDGPURegisterBankInfo.cpp
The file was modified/llvm/trunk/test/CodeGen/AMDGPU/GlobalISel/regbankselect-select.mir (diff)llvm.src/test/CodeGen/AMDGPU/GlobalISel/regbankselect-select.mir