[llvm-commits] [PATCH] Make Inits FoldingSetNodes

David A. Greene greened at obbligato.org
Mon Jul 18 11:13:04 PDT 2011


Jakob Stoklund Olesen <stoklund at 2pi.dk> writes:

> Note that I didn't change all of the RecTy pointers into const
> pointers. I don't think it is necessary since the RecTy classes don't
> have any public mutators.

Ok.

> You could do the same for the Inits - It would require much fewer source changes.

Having the "const" there helped catch a few bugs so I'm inclined to keep
it.  I've separated that bit of the patch out anyway.

                                 -Dave



More information about the llvm-commits mailing list