[PATCH] D22519: Invariant start/end intrinsics overloaded for address space

[Anna Thomas via llvm-commits]

anna added a comment.

In https://reviews.llvm.org/D22519#493183, @anna wrote:

> Committed llvm changes in https://reviews.llvm.org/rL276447
>
> Commited clang changes in: https://reviews.llvm.org/rL276448
>
> No clang failures.


The clang build break was due to the absence of overloaded type in the function call for getting the invariant intrinsic in `clang::CodeGen::CodeGenFunction::EmitDeclInvariant`. Clang test failures are due to the insertion of llvm.invariant.start calls after the constructor. This is turned on only at O1. 
Discussed with Artur offline and the fix being simple and similar to the llvm invariant intrinsic change in `AutoUpgrade`, is checked in to clang directly. All clang and llvm buildbots pass.


https://reviews.llvm.org/D22519