search for: nfci

Hi, I got a forced update when pulling today. If I merge master to a local branch, I get a bunch of add/add conflicts. This same commit exists under several hashes: https://github.com/llvm-project/llvm-project-20170507/commit/687841777ef505 https://github.com/llvm-project/llvm-project-20170507/commit/74725885552 Did someone push -f to the monorepo after doing branch surgery? Maybe there was a

...ld you take a look please? Thanks, Jay. On Tue, 10 Dec 2019 at 06:19, Johannes Doerfert via Phabricator via llvm-commits <llvm-commits at lists.llvm.org> wrote: > > This revision was automatically updated to reflect the committed changes. > Closed by commit rGeb3e81f43f01: [OpenMP][NFCI] Introduce llvm/IR/OpenMPConstants.h (authored by jdoerfert). > > Changed prior to commit: > https://reviews.llvm.org/D69853?vs=232987&id=233003#toc > > Repository: > rG LLVM Github Monorepo > > CHANGES SINCE LAST ACTION > https://reviews.llvm.org/D69853/new/ &g...

...Tue, 10 Dec 2019 at 06:19, Johannes Doerfert via Phabricator via > > llvm-commits <llvm-commits at lists.llvm.org> wrote: > > > > > > This revision was automatically updated to reflect the committed > changes. > > > Closed by commit rGeb3e81f43f01: [OpenMP][NFCI] Introduce > llvm/IR/OpenMPConstants.h (authored by jdoerfert). > > > > > > Changed prior to commit: > > > https://reviews.llvm.org/D69853?vs=232987&id=233003#toc > > > > > > Repository: > > > rG LLVM Github Monorepo > > > &g...

...habricator via > > > > llvm-commits <llvm-commits at lists.llvm.org> wrote: > > > > > > > > > > This revision was automatically updated to reflect the committed > > > changes. > > > > > Closed by commit rGeb3e81f43f01: [OpenMP][NFCI] Introduce > > > llvm/IR/OpenMPConstants.h (authored by jdoerfert). > > > > > > > > > > Changed prior to commit: > > > > > https://reviews.llvm.org/D69853?vs=232987&id=233003#toc > > > > > > > > > > Repository...

...y pass manager... +3% r247264: Enable GlobalsAA by default. +1% 6. r247674: [GlobalsAA] Disable globals-aa by default. -1% 7. r248638: [SCEV] Reapply 'Teach isLoopBackedgeGuardedByCond to exploit trip counts'. +2% 8. r249802: [SCEV] Call `StrengthenNoWrapFlags` after `GroupByComplexity`; NFCI. +4% 9. r250157: [GlobalsAA] Turn GlobalsAA on again by default. +1% 10. r251049: [SCEV] Mark AddExprs as nsw or nuw if legal. +23% 11. No data 12. r259252: AttributeSetImpl: Summarize existing function attributes in a bitset. -1% r259256: Add LoopSimplifyCFG pass. -2% 13. r262250: Enable LoopL...

...th? Register seems to be delegating to MCRegister without owning any new additional responsibilities. Register provides everything related to vregs and stack slots. Since I last looked at these files it seems a bit more of the number space definitions have been moved to MCRegister by 16dae81edc2 [NFCI] Cleanup range checks in Register/MCRegister I'd intentionally left the vreg bit in Register because vregs weren't a thing to MCRegister and that range was already excluded by the existing code but it makes sense to define the whole number space there so long as MCRegister continues to not...

...47264: Enable GlobalsAA by default. +1% >> 6. r247674: [GlobalsAA] Disable globals-aa by default. -1% >> 7. r248638: [SCEV] Reapply 'Teach isLoopBackedgeGuardedByCond to exploit trip counts'. +2% >> 8. r249802: [SCEV] Call `StrengthenNoWrapFlags` after `GroupByComplexity`; NFCI. +4% >> 9. r250157: [GlobalsAA] Turn GlobalsAA on again by default. +1% >> 10. r251049: [SCEV] Mark AddExprs as nsw or nuw if legal. +23% >> 11. No data >> 12. r259252: AttributeSetImpl: Summarize existing function attributes in a bitset. -1% >> r259256: Add LoopSi...

...o be delegating to MCRegister without owning any new additional responsibilities. > > Register provides everything related to vregs and stack slots. > Since I last looked at these files it seems a bit more of the number space definitions have been moved to MCRegister by > 16dae81edc2 [NFCI] Cleanup range checks in Register/MCRegister > I'd intentionally left the vreg bit in Register because vregs weren't a thing to MCRegister and that range was already excluded by the existing code but it makes sense to define the whole number space there so long as MCRegister continues to...

...GlobalsAA by default. +1% > > 6. r247674: [GlobalsAA] Disable globals-aa by default. -1% > > 7. r248638: [SCEV] Reapply 'Teach isLoopBackedgeGuardedByCond to exploit > trip counts'. +2% > > 8. r249802: [SCEV] Call `StrengthenNoWrapFlags` after > `GroupByComplexity`; NFCI. +4% > > 9. r250157: [GlobalsAA] Turn GlobalsAA on again by default. +1% > > 10. r251049: [SCEV] Mark AddExprs as nsw or nuw if legal. +23% > > 11. No data > > 12. r259252: AttributeSetImpl: Summarize existing function attributes in > a bitset. -1% > > r259256: A...

...to MCRegister without owning any new additional responsibilities. >> >> Register provides everything related to vregs and stack slots. >> Since I last looked at these files it seems a bit more of the number space definitions have been moved to MCRegister by >> 16dae81edc2 [NFCI] Cleanup range checks in Register/MCRegister >> I'd intentionally left the vreg bit in Register because vregs weren't a thing to MCRegister and that range was already excluded by the existing code but it makes sense to define the whole number space there so long as MCRegister continue...

On Tue, Sep 29, 2020 at 9:08 AM Quentin Colombet <qcolombet at apple.com> wrote: > Hi, > > Register can represent virtual or physical registers. > MCRegister can only represent physical registers. > That's what I thought, but MCRegister has some stack slot APIs. > Eventually all Register instances are replaced by a MCRegister. > What happens in that case to the

<div class="socmaildefaultfont" dir="ltr" style="font-family:Arial, Helvetica, sans-serif;font-size:10pt" ><div dir="ltr" ><font face="AppleSystemUIFont" size="3" >LoopUnrollAndJamPass is currently a loop pass. It is added in a LPM with only itself.</font><br><font face="AppleSystemUIFont"