efriedma-quic wrote:
The timing of this is sort of unfortunate with #132252; the merge conflict
there is going to be very painful. @jthackray , thoughts?
https://github.com/llvm/llvm-project/pull/132060
___
cfe-commits mailing list
cfe-commits@lists.
https://github.com/AaronBallman edited
https://github.com/llvm/llvm-project/pull/132060
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul closed
https://github.com/llvm/llvm-project/pull/132060
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AaronBallman approved this pull request.
Changes LGTM though I've added @efriedma-quic as a codegen code owner just in
case he's got different opinions.
https://github.com/llvm/llvm-project/pull/132060
___
cfe-commits mailing list
c
https://github.com/jthackray approved this pull request.
LGTM.
https://github.com/llvm/llvm-project/pull/132060
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jthackray edited
https://github.com/llvm/llvm-project/pull/132060
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jthackray wrote:
> The timing of this is sort of unfortunate with #132252; the merge conflict
> there is going to be very painful. @jthackray , thoughts?
@efriedma-quic I think I can update my diff after this is merged; yes, might
take a while to resolve the diffs.
https://github.com/llvm/llv
https://github.com/jurahul created
https://github.com/llvm/llvm-project/pull/132060
- Use `Intrinsic::` directly instead of `llvm::Intrinsic::`.
- Eliminate redundant `nullptr` for some `CreateIntrinsic` calls.
- Eliminate redundant `ArrayRef` casts.
- Use C++17 structured binding instead of `st
