FailedChanges

Summary

  1. Set hidden attribute on lprofMergeValueProfData Summary: The changes in https://reviews.llvm.org/D44847 cause load time failure due to lprofMergeValueProfData in Android libs enabled with profile generation: "dlopen failed: cannot locate symbol "lprofMergeValueProfData" referenced by..." Marking lprofMergeValueProfData as hidden so the correct in-module definition is picked by the linker. Reviewers: davidxl Reviewed By: davidxl Subscribers: efriedma, xur, davidxl, llvm-commits Differential Revision: https://reviews.llvm.org/D55893
  2. Further relax restriction in tests to include where "-E" and "-S" must appear. Also updated a few instances of "-emit-llvm-bc" and "-emit-obj" that were missed in the previous change.
  3. [ThinLTO] Detect partially split modules during the thin link Summary: The changes to disable LTO unit splitting by default (r350949) and detect inconsistently split LTO units (r350948) are causing some crashes when the inconsistency is detected in multiple threads simultaneously. Fix that by having the code always look for the inconsistently split LTO units during the thin link, by checking for the presence of type tests recorded in the summaries. Modify test added in r350948 to remove single threading required to fix a bot failure due to this issue (and some debugging options added in the process of diagnosing it). Reviewers: pcc Subscribers: mehdi_amini, inglorion, eraman, steven_wu, dexonsmith, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D57561
  4. [libFuzzer] better stats for the fork mode
  5. [CMake] Fix ability to use LLVM_ENABLE_PROJECTS with LLVM_EXTERNAL_PROJECTS LLVM r353148, changed the circumstances in which the project source directory variables are created to only create them for LLVM projects. This patch initializes the directory variables for projects specified in `LLVM_EXTERNAL_PROJECTS` as well.
  6. [InstCombine] Add todos for possible atomicrmw transforms
  7. Canonicalize all integer "idempotent" atomicrmw ops For "idempotent" atomicrmw instructions which we can't simply turn into load, canonicalize the operation and constant. This reduces the matching needed elsewhere in the optimizer, but doesn't directly impact codegen. For any architecture where OR/Zero is not a good default choice, you can extend the AtomicExpand lowerIdempotentRMWIntoFencedLoad mechanism. I reviewed X86 to make sure this works well, haven't audited other backends. Differential Revision: https://reviews.llvm.org/D58244
  8. Stop enabling clang-tools-extra automatically when clang is in LLVM_ENABLE_PROJECTS If you want to build clang-tools-extra with monorepo, just add it to LLVM_ENABLE_PROJECTS like with other projects. See also "Separating clang-tools-extra from clang in LLVM_ENABLE_PROJECTS" on cfe-dev. Differential Revision: https://reviews.llvm.org/D58157
  9. [CodeGenObjC] Emit [[X alloc] init] as objc_alloc_init(X) when available This provides a code size win on the caller side, since the init message send is done in the runtime function. rdar://44987038 Differential revision: https://reviews.llvm.org/D57936
  10. Optional specialization for trivially copyable types, part2
  11. [X86] Add clang support for X86 flag output parameters. Summary: Add frontend support and expected flags for X86 inline assembly flag parameters. Reviewers: craig.topper, rnk, echristo Subscribers: eraman, nickdesaulniers, void, llvm-commits Differential Revision: https://reviews.llvm.org/D57394
  12. Recommit Optional specialization for trivially copyable types Unfortunately the original code gets misscompiled by GCC (at least 8.1), this is a tentative workaround using std::memcpy instead of inplace new for trivially copyable types. I'll revert if it breaks. Original revision: https://reviews.llvm.org/D57097
  13. Teach instcombine about remaining "idempotent" atomicrmw types Expand on Quentin's r353471 patch which converts some atomicrmws into loads. Handle remaining operation types, and fix a slight bug. Atomic loads are required to have alignment. Since this was within the InstCombine fixed point, somewhere else in InstCombine was adding alignment before the verifier saw it, but still, we should fix. Terminology wise, I'm using the "idempotent" naming that is used for the same operations in AtomicExpand and X86ISelLoweringInfo. Once this lands, I'll add similar tests for AtomicExpand, and move the pattern match function to a common location. In the review, there was seemingly consensus that "idempotent" was slightly incorrect for this context. Once we setle on a better name, I'll update all uses at once. Differential Revision: https://reviews.llvm.org/D58242
  14. Support: use internal `call_once` on PPC64le Always use the internal `call_once` for PPC64le. This is needed to support the Swift toolchain on PPC64le. Patch by Sarvesh Tamba!
  15. [llvm-ar] Implement the P modifier. Summary: GNU ar has a `P` modifier that changes filename comparisons to use full paths instead of the basename. As noted in the GNU docs, regular archives are not created with full path names, so P is used to deal with archives created by other archive programs (e.g. see the updated `absolute-paths.test` test case). Since thin archives use full path names -- paths are relative to the archive -- it seems very error prone to not imply P when dealing with thin archives, so P is implied in those cases. (I think this is a deviation from GNU ar that makes sense). This fixes PR37436 via https://github.com/ClangBuiltLinux/linux/issues/33. Reviewers: mstorsjo, pcc, ruiu, davide, david2050, rnk Subscribers: tpimh, llvm-commits, nickdesaulniers Tags: #llvm Differential Revision: https://reviews.llvm.org/D57927
  16. [X86] cleanup inline asm register generation. NFCI.
  17. [SystemZ] Do not emit VEXTEND or VROUND nodes without vector support. Review: Ulrich Weigand https://reviews.llvm.org/D58240
  18. [Tests] Add tests for all idemptotent atomicrmws Base for a followup patch to strengthen the InstCombine transform, and then integrate the ExpandAtomics logic.
  19. [Sema] Fix a regression introduced in "[AST][Sema] Remove CallExpr::setNumArgs" D54902 removed CallExpr::setNumArgs in preparation of tail-allocating the arguments of CallExpr. It did this by allocating storage for max(number of arguments, number of parameters in the prototype). The temporarily nulled arguments however causes issues in BuildResolvedCallExpr when typo correction is done just after the creation of the call expression. This was unfortunately missed by the tests /: To fix this, delay setting the number of arguments to max(number of arguments, number of parameters in the prototype) until we are ready for it. It would be nice to have this encapsulated in CallExpr but this is the best I can come up with under the constraint that we cannot add anything the CallExpr. Fixes PR40286. Differential Revision: https://reviews.llvm.org/D57948 Reviewed By: aaron.ballman
  20. [X86][AVX] Add PR40730 test case
  21. Refine ArgPromotion metadata handling Summary: In r353537 we now copy all metadata to the new function, with the old being removed when the old function is eliminated. In some cases the old function is dropped to a declaration (seems to only occur with the old PM). Go ahead and clear all metadata from the old function to handle that case, since verification will complain otherwise. This is consistent with what was being done for debug metadata before r353537. Reviewers: davidxl, uabelho Subscribers: jdoerfert, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D58215
  22. [LoopUnrollPeel] Add case where we should forget the peeled loop from SCEV. The test case requires the peeled loop to be forgotten after peeling, even though it does not have a parent. When called via the unroller, SE->forgetTopmostLoop is also called, so the test case would also pass without any SCEV invalidation, but peelLoop is exposed as utility function. Also, in the test case, simplifyLoop will make changes, removing the loop from SCEV, but it is better to not rely on this behavior. Reviewers: sanjoy, mkazantsev Reviewed By: mkazantsev Tags: #llvm Differential Revision: https://reviews.llvm.org/D58192
  23. [RISCV][NFC] Add RV64I CHECK lines to inline-asm.ll test
  24. [ASTImporter] Check visibility/linkage of functions and variables Summary: During import of a global variable with external visibility the lookup will find variables (with the same name) but with static visibility. Clearly, we cannot put them into the same redecl chain. The same is true in case of functions. In this fix we filter the lookup results and consider only those which have the same visibility as the decl we currently import. We consider two decls in two anonymous namsepaces to have the same visibility only if they are imported from the very same translation unit. Reviewers: a_sidorin, shafik, a.sidorin Reviewed By: shafik Subscribers: jdoerfert, balazske, rnkovacs, dkrupp, Szelethus, gamesh411, cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D57232
  25. Reapply [VFS] Allow multiple RealFileSystem instances with independent CWDs. This reverts commit r351091. The original mac breakages are addressed by ensuring the root directory we're working from is fully symlink-resolved before starting. Differential Revision: https://reviews.llvm.org/D58169
  26. [MIPS GlobalISel] Select phi instruction for integers Select G_PHI for integers for MIPS32. Differential Revision: https://reviews.llvm.org/D58183
Revision 354064 by apazos:
Set hidden attribute on lprofMergeValueProfData

Summary:
The changes in https://reviews.llvm.org/D44847 cause load time failure
due to lprofMergeValueProfData in Android libs enabled with profile generation:

"dlopen failed: cannot locate symbol "lprofMergeValueProfData" referenced by..."

Marking lprofMergeValueProfData as hidden so the correct in-module definition
is picked by the linker.

Reviewers: davidxl

Reviewed By: davidxl

Subscribers: efriedma, xur, davidxl, llvm-commits

Differential Revision: https://reviews.llvm.org/D55893
Change TypePath in RepositoryPath in Workspace
The file was modified/compiler-rt/trunk/lib/profile/InstrProfilingMergeFile.c (diff)compiler-rt.src/lib/profile/InstrProfilingMergeFile.c
The file was added/compiler-rt/trunk/test/profile/Inputs/instrprof-value-prof-visibility.ccompiler-rt.src/test/profile/Inputs/instrprof-value-prof-visibility.c
The file was added/compiler-rt/trunk/test/profile/Linux/instrprof-value-prof-visibility.testcompiler-rt.src/test/profile/Linux/instrprof-value-prof-visibility.test
Revision 354063 by dyung:
Further relax restriction in tests to include where "-E" and "-S" must appear.

Also updated a few instances of "-emit-llvm-bc" and "-emit-obj" that were missed in the previous change.
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/test/Driver/openmp-offload.c (diff)clang.src/test/Driver/openmp-offload.c
Revision 354062 by tejohnson:
[ThinLTO] Detect partially split modules during the thin link

Summary:
The changes to disable LTO unit splitting by default (r350949) and
detect inconsistently split LTO units (r350948) are causing some crashes
when the inconsistency is detected in multiple threads simultaneously.
Fix that by having the code always look for the inconsistently split
LTO units during the thin link, by checking for the presence of type
tests recorded in the summaries.

Modify test added in r350948 to remove single threading required to fix
a bot failure due to this issue (and some debugging options added in the
process of diagnosing it).

Reviewers: pcc

Subscribers: mehdi_amini, inglorion, eraman, steven_wu, dexonsmith, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D57561
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/LTO/LTO.h (diff)llvm.src/include/llvm/LTO/LTO.h
The file was modified/llvm/trunk/lib/LTO/LTO.cpp (diff)llvm.src/lib/LTO/LTO.cpp
The file was modified/llvm/trunk/lib/Transforms/IPO/LowerTypeTests.cpp (diff)llvm.src/lib/Transforms/IPO/LowerTypeTests.cpp
The file was modified/llvm/trunk/lib/Transforms/IPO/WholeProgramDevirt.cpp (diff)llvm.src/lib/Transforms/IPO/WholeProgramDevirt.cpp
The file was modified/llvm/trunk/test/ThinLTO/X86/cfi-devirt.ll (diff)llvm.src/test/ThinLTO/X86/cfi-devirt.ll
Revision 354061 by kcc:
[libFuzzer] better stats for 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
Revision 354060 by cbieneman:
[CMake] Fix ability to use LLVM_ENABLE_PROJECTS with LLVM_EXTERNAL_PROJECTS

LLVM r353148, changed the circumstances in which the project source directory variables are created to only create them for LLVM projects. This patch initializes the directory variables for projects specified in `LLVM_EXTERNAL_PROJECTS` as well.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/CMakeLists.txt (diff)llvm.src/CMakeLists.txt
Revision 354059 by reames:
[InstCombine] Add todos for possible atomicrmw transforms
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Transforms/InstCombine/InstCombineAtomicRMW.cpp (diff)llvm.src/lib/Transforms/InstCombine/InstCombineAtomicRMW.cpp
Revision 354058 by reames:
Canonicalize all integer "idempotent" atomicrmw ops

For "idempotent" atomicrmw instructions which we can't simply turn into load, canonicalize the operation and constant. This reduces the matching needed elsewhere in the optimizer, but doesn't directly impact codegen.

For any architecture where OR/Zero is not a good default choice, you can extend the AtomicExpand lowerIdempotentRMWIntoFencedLoad mechanism. I reviewed X86 to make sure this works well, haven't audited other backends.

Differential Revision: https://reviews.llvm.org/D58244
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Transforms/InstCombine/InstCombineAtomicRMW.cpp (diff)llvm.src/lib/Transforms/InstCombine/InstCombineAtomicRMW.cpp
The file was modified/llvm/trunk/test/Transforms/InstCombine/atomicrmw.ll (diff)llvm.src/test/Transforms/InstCombine/atomicrmw.ll
Revision 354057 by nico:
Stop enabling clang-tools-extra automatically when clang is in LLVM_ENABLE_PROJECTS

If you want to build clang-tools-extra with monorepo, just add it to
LLVM_ENABLE_PROJECTS like with other projects.

See also "Separating clang-tools-extra from clang in LLVM_ENABLE_PROJECTS"
on cfe-dev.

Differential Revision: https://reviews.llvm.org/D58157
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/CMakeLists.txt (diff)llvm.src/CMakeLists.txt
The file was modified/llvm/trunk/docs/GettingStarted.rst (diff)llvm.src/docs/GettingStarted.rst
Revision 354056 by epilk:
[CodeGenObjC] Emit [[X alloc] init] as objc_alloc_init(X) when available

This provides a code size win on the caller side, since the init
message send is done in the runtime function.

rdar://44987038

Differential revision: https://reviews.llvm.org/D57936
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/include/clang/Basic/ObjCRuntime.h (diff)clang.src/include/clang/Basic/ObjCRuntime.h
The file was modified/cfe/trunk/lib/CodeGen/CGObjC.cpp (diff)clang.src/lib/CodeGen/CGObjC.cpp
The file was modified/cfe/trunk/lib/CodeGen/CodeGenFunction.h (diff)clang.src/lib/CodeGen/CodeGenFunction.h
The file was modified/cfe/trunk/lib/CodeGen/CodeGenModule.h (diff)clang.src/lib/CodeGen/CodeGenModule.h
The file was added/cfe/trunk/test/CodeGenObjC/objc-alloc-init.mclang.src/test/CodeGenObjC/objc-alloc-init.m
Revision 354055 by serge_sans_paille:
Optional specialization for trivially copyable types, part2
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/ADT/Optional.h (diff)llvm.src/include/llvm/ADT/Optional.h
Revision 354053 by niravd:
[X86] Add clang support for X86 flag output parameters.

Summary:
Add frontend support and expected flags for X86 inline assembly flag
parameters.

Reviewers: craig.topper, rnk, echristo

Subscribers: eraman, nickdesaulniers, void, llvm-commits

Differential Revision: https://reviews.llvm.org/D57394
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/lib/Basic/Targets/X86.cpp (diff)clang.src/lib/Basic/Targets/X86.cpp
The file was added/cfe/trunk/test/CodeGen/inline-asm-x86-flag-output.cclang.src/test/CodeGen/inline-asm-x86-flag-output.c
The file was modified/cfe/trunk/test/Preprocessor/predefined-win-macros.c (diff)clang.src/test/Preprocessor/predefined-win-macros.c
The file was added/cfe/trunk/test/Preprocessor/x86_asm_flag_output.cclang.src/test/Preprocessor/x86_asm_flag_output.c
Revision 354051 by serge_sans_paille:
Recommit Optional specialization for trivially copyable types

Unfortunately the original code gets misscompiled by GCC (at least 8.1),
this is a tentative workaround using std::memcpy instead of inplace new
for trivially copyable types. I'll revert if it breaks.

Original revision: https://reviews.llvm.org/D57097
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/ADT/Optional.h (diff)llvm.src/include/llvm/ADT/Optional.h
The file was modified/llvm/trunk/unittests/ADT/OptionalTest.cpp (diff)llvm.src/unittests/ADT/OptionalTest.cpp
Revision 354046 by reames:
Teach instcombine about remaining "idempotent" atomicrmw types

Expand on Quentin's r353471 patch which converts some atomicrmws into loads. Handle remaining operation types, and fix a slight bug. Atomic loads are required to have alignment. Since this was within the InstCombine fixed point, somewhere else in InstCombine was adding alignment before the verifier saw it, but still, we should fix.

Terminology wise, I'm using the "idempotent" naming that is used for the same operations in AtomicExpand and X86ISelLoweringInfo. Once this lands, I'll add similar tests for AtomicExpand, and move the pattern match function to a common location. In the review, there was seemingly consensus that "idempotent" was slightly incorrect for this context.  Once we setle on a better name, I'll update all uses at once.

Differential Revision: https://reviews.llvm.org/D58242
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Transforms/InstCombine/InstCombineAtomicRMW.cpp (diff)llvm.src/lib/Transforms/InstCombine/InstCombineAtomicRMW.cpp
The file was modified/llvm/trunk/test/Transforms/InstCombine/atomicrmw.ll (diff)llvm.src/test/Transforms/InstCombine/atomicrmw.ll
Revision 354045 by Saleem Abdulrasool:
Support: use internal `call_once` on PPC64le

Always use the internal `call_once` for PPC64le.  This is needed to
support the Swift toolchain on PPC64le.

Patch by Sarvesh Tamba!
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/Support/Threading.h (diff)llvm.src/include/llvm/Support/Threading.h
Revision 354044 by rupprecht:
[llvm-ar] Implement the P modifier.

Summary:
GNU ar has a `P` modifier that changes filename comparisons to use full paths instead of the basename. As noted in the GNU docs, regular archives are not created with full path names, so P is used to deal with archives created by other archive programs (e.g. see the updated `absolute-paths.test` test case).

Since thin archives use full path names -- paths are relative to the archive -- it seems very error prone to not imply P when dealing with thin archives, so P is implied in those cases. (I think this is a deviation from GNU ar that makes sense).

This fixes PR37436 via https://github.com/ClangBuiltLinux/linux/issues/33.

Reviewers: mstorsjo, pcc, ruiu, davide, david2050, rnk

Subscribers: tpimh, llvm-commits, nickdesaulniers

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D57927
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/test/tools/llvm-ar/absolute-paths.test (diff)llvm.src/test/tools/llvm-ar/absolute-paths.test
The file was added/llvm/trunk/test/tools/llvm-ar/full-path-option.testllvm.src/test/tools/llvm-ar/full-path-option.test
The file was modified/llvm/trunk/tools/llvm-ar/llvm-ar.cpp (diff)llvm.src/tools/llvm-ar/llvm-ar.cpp
Revision 354042 by niravd:
[X86] cleanup inline asm register generation. NFCI.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/X86/X86ISelLowering.cpp (diff)llvm.src/lib/Target/X86/X86ISelLowering.cpp
Revision 354039 by jonpa:
[SystemZ] Do not emit VEXTEND or VROUND nodes without vector support.

Review: Ulrich Weigand
https://reviews.llvm.org/D58240
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 added/llvm/trunk/test/CodeGen/SystemZ/fp-conv-18.mirllvm.src/test/CodeGen/SystemZ/fp-conv-18.mir
Revision 354036 by reames:
[Tests] Add tests for all idemptotent atomicrmws

Base for a followup patch to strengthen the InstCombine transform, and then integrate the ExpandAtomics logic.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/test/Transforms/InstCombine/atomicrmw.ll (diff)llvm.src/test/Transforms/InstCombine/atomicrmw.ll
Revision 354035 by brunoricci:
[Sema] Fix a regression introduced in "[AST][Sema] Remove CallExpr::setNumArgs"

D54902 removed CallExpr::setNumArgs in preparation of tail-allocating the
arguments of CallExpr. It did this by allocating storage for
max(number of arguments, number of parameters in the prototype). The
temporarily nulled arguments however causes issues in BuildResolvedCallExpr
when typo correction is done just after the creation of the call expression.

This was unfortunately missed by the tests /:

To fix this, delay setting the number of arguments to
max(number of arguments, number of parameters in the prototype) until we are
ready for it. It would be nice to have this encapsulated in CallExpr but this
is the best I can come up with under the constraint that we cannot add
anything the CallExpr.

Fixes PR40286.

Differential Revision: https://reviews.llvm.org/D57948

Reviewed By: aaron.ballman
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/lib/Sema/SemaExpr.cpp (diff)clang.src/lib/Sema/SemaExpr.cpp
The file was modified/cfe/trunk/test/Sema/typo-correction.c (diff)clang.src/test/Sema/typo-correction.c
Revision 354034 by rksimon:
[X86][AVX] Add PR40730 test case
Change TypePath in RepositoryPath in Workspace
The file was added/llvm/trunk/test/CodeGen/X86/pr40730.llllvm.src/test/CodeGen/X86/pr40730.ll
Revision 354032 by tejohnson:
Refine ArgPromotion metadata handling

Summary:
In r353537 we now copy all metadata to the new function, with the old
being removed when the old function is eliminated. In some cases the old
function is dropped to a declaration (seems to only occur with the old
PM). Go ahead and clear all metadata from the old function to handle that
case, since verification will complain otherwise. This is consistent
with what was being done for debug metadata before r353537.

Reviewers: davidxl, uabelho

Subscribers: jdoerfert, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D58215
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Transforms/IPO/ArgumentPromotion.cpp (diff)llvm.src/lib/Transforms/IPO/ArgumentPromotion.cpp
The file was added/llvm/trunk/test/Transforms/ArgumentPromotion/dbg2.llllvm.src/test/Transforms/ArgumentPromotion/dbg2.ll
Revision 354031 by fhahn:
[LoopUnrollPeel] Add case where we should forget the peeled loop from SCEV.

The test case requires the peeled loop to be forgotten after peeling,
even though it does not have a parent. When called via the unroller,
SE->forgetTopmostLoop is also called, so the test case would also pass
without any SCEV invalidation, but peelLoop is exposed as utility
function. Also, in the test case, simplifyLoop will make changes,
removing the loop from SCEV, but it is better to not rely on this
behavior.

Reviewers: sanjoy, mkazantsev

Reviewed By: mkazantsev

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D58192
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Transforms/Utils/LoopUnrollPeel.cpp (diff)llvm.src/lib/Transforms/Utils/LoopUnrollPeel.cpp
The file was added/llvm/trunk/test/Transforms/LoopUnroll/peel-loop-scev-invalidate.llllvm.src/test/Transforms/LoopUnroll/peel-loop-scev-invalidate.ll
Revision 354028 by asb:
[RISCV][NFC] Add RV64I CHECK lines to inline-asm.ll test
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/test/CodeGen/RISCV/inline-asm.ll (diff)llvm.src/test/CodeGen/RISCV/inline-asm.ll
Revision 354027 by martong:
[ASTImporter] Check visibility/linkage of functions and variables

Summary:
During import of a global variable with external visibility the lookup
will find variables (with the same name) but with static visibility.
Clearly, we cannot put them into the same redecl chain.  The same is
true in case of functions.  In this fix we filter the lookup results and
consider only those which have the same visibility as the decl we
currently import.

We consider two decls in two anonymous namsepaces to have the same
visibility only if they are imported from the very same translation
unit.

Reviewers: a_sidorin, shafik, a.sidorin

Reviewed By: shafik

Subscribers: jdoerfert, balazske, rnkovacs, dkrupp, Szelethus, gamesh411, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D57232
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/include/clang/AST/ASTImporter.h (diff)clang.src/include/clang/AST/ASTImporter.h
The file was modified/cfe/trunk/lib/AST/ASTImporter.cpp (diff)clang.src/lib/AST/ASTImporter.cpp
The file was modified/cfe/trunk/unittests/AST/ASTImporterTest.cpp (diff)clang.src/unittests/AST/ASTImporterTest.cpp
Revision 354026 by sammccall:
Reapply [VFS] Allow multiple RealFileSystem instances with independent CWDs.

This reverts commit r351091.
The original mac breakages are addressed by ensuring the root directory
we're working from is fully symlink-resolved before starting.

Differential Revision: https://reviews.llvm.org/D58169
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/Support/VirtualFileSystem.h (diff)llvm.src/include/llvm/Support/VirtualFileSystem.h
The file was modified/llvm/trunk/lib/Support/VirtualFileSystem.cpp (diff)llvm.src/lib/Support/VirtualFileSystem.cpp
The file was modified/llvm/trunk/unittests/Support/VirtualFileSystemTest.cpp (diff)llvm.src/unittests/Support/VirtualFileSystemTest.cpp
Revision 354025 by petar.avramovic:
[MIPS GlobalISel] Select phi instruction for integers

Select G_PHI for integers for MIPS32.

Differential Revision: https://reviews.llvm.org/D58183
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/Mips/MipsInstructionSelector.cpp (diff)llvm.src/lib/Target/Mips/MipsInstructionSelector.cpp
The file was modified/llvm/trunk/lib/Target/Mips/MipsLegalizerInfo.cpp (diff)llvm.src/lib/Target/Mips/MipsLegalizerInfo.cpp
The file was added/llvm/trunk/test/CodeGen/Mips/GlobalISel/instruction-select/phi.mirllvm.src/test/CodeGen/Mips/GlobalISel/instruction-select/phi.mir
The file was added/llvm/trunk/test/CodeGen/Mips/GlobalISel/legalizer/phi.mirllvm.src/test/CodeGen/Mips/GlobalISel/legalizer/phi.mir
The file was added/llvm/trunk/test/CodeGen/Mips/GlobalISel/llvm-ir/phi.llllvm.src/test/CodeGen/Mips/GlobalISel/llvm-ir/phi.ll
The file was added/llvm/trunk/test/CodeGen/Mips/GlobalISel/regbankselect/phi.mirllvm.src/test/CodeGen/Mips/GlobalISel/regbankselect/phi.mir