[llvm] r242173 - MachineRegisterInfo: Remove UsedPhysReg infrastructure

Tom Stellard via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 13 16:52:25 PDT 2015


On Thu, Aug 13, 2015 at 03:53:06PM -0700, Quentin Colombet via llvm-commits wrote:
> 
> > On Aug 13, 2015, at 3:20 PM, Marek Olšák <maraeo at gmail.com> wrote:
> > 
> > On Fri, Aug 14, 2015 at 12:09 AM, Quentin Colombet <qcolombet at apple.com> wrote:
> >> Hi Marek,
> >> 
> >> I will take care of it, Matthias is on vacation.
> >> 
> >> Sorry for not noticing earlier.
> >> 
> >>> On Aug 13, 2015, at 3:06 PM, Marek Olšák via llvm-commits <llvm-commits at lists.llvm.org> wrote:
> >>> 
> >>> Hi,
> >>> 
> >>> Just a reminder: We really need to revert this in the 3.7 branch ASAP.
> >> 
> >> Should I revert on trunk as well?
> > 
> > Hi Quentin,
> > 
> > That depends on whether or not the regression will be fixed there in
> > the near future.
> 
> Ok, I’ll wait for Matthias to look into it before deciding.
> 
> My guess is that the fix will likely be in the target specific part, not here.
> 
> Anyway, I have reverted this on release_37: r244976.
> 

I opened an audit for this commit, and I think I've spotted the problem:

http://reviews.llvm.org/rL242173

-Tom

> Cheers,
> -Quentin
> 
> > 
> > Marek
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits


More information about the llvm-commits mailing list