[PATCH] D139229: Use CTAD on llvm::SaveAndRestore

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGabf0c6c0c0b7: Use CTAD on llvm::SaveAndRestore (authored by jansvoboda11). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https:

[PATCH] D139229: Use CTAD on llvm::SaveAndRestore

dblaikie accepted this revision. dblaikie added a comment. This revision is now accepted and ready to land. Sounds good to me, thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D139229/new/ https://reviews.llvm.org/D139229 _

[PATCH] D139229: Use CTAD on llvm::SaveAndRestore

kazu added a comment. In D139229#3967666 , @jansvoboda11 wrote: > Sorry for the noise, didn't realize this will trigger Herald rules. Wanted to > check with @kazu if this looks fine in principle and if it should be split up > into multiple commits (e.g

[PATCH] D139229: Use CTAD on llvm::SaveAndRestore

jansvoboda11 added a comment. Herald added a subscriber: jsetoain. Sorry for the noise, didn't realize this will trigger Herald rules. Wanted to check with @kazu if this looks fine in principle and if it should be split up into multiple commits (e.g. per project). Repository: rG LLVM Github

[PATCH] D139229: Use CTAD on llvm::SaveAndRestore

jansvoboda11 created this revision. jansvoboda11 added a reviewer: kazu. Herald added subscribers: Moerafaat, zero9178, steakhal, bzcheeseman, sdasgup3, carlosgalvezp, jeroen.dobbelaere, wenzhicui, wrengr, cota, teijeong, rdzhabarov, tatianashp, ThomasRaoux, msifontes, jurahul, Kayjukh, grosul1,