vtjnash wrote:

> removed the TypePartitionTy == AI.getAllocatedType() check

The request nikic was making, on my behalf, was to pull this entirely outside 
of the `AI.getAllocatedType()` branch of the function, since I'm moving towards 
being able to delete that branch, as he mentioned. That said, I don't think 
that change is actually necessary for this PR. As long as you're confident in 
your tests here representing a cross-section of what this optimization should 
be able to do, then I think it will provide good coverage to ensure any future 
changes preserve the intent here.


https://github.com/llvm/llvm-project/pull/165159
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to