[llvm] r302416 - [ARM] Clear the constant pool cache on explicit .ltorg directives

Martin Storsjö via llvm-commits llvm-commits at lists.llvm.org
Mon May 22 02:26:29 PDT 2017


Hi James,

On Mon, 22 May 2017, James Molloy wrote:

> Hi Martin,
> Apologies, I have reverted this in r303536. I had to revert the
> underlying r286006, which is what your patch was fixing. r286006 caused
> PR32825 which wasn't addressed.

In a quick test I did right now, PR32825 seems to have been fixed by my 
fix in r302416 as well. (It's the exact same issue I had, which I fixed in 
r302416 although I never made a PR out of the issue.) So with the current 
trunk prior to your reverts, you could just have closed PR32825 as fixed, 
no?

// Martin


More information about the llvm-commits mailing list