[PATCH] D16250: Respect bound archs, even when they don't alter the toolchain.

Justin Lebar via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 15 19:27:16 PST 2016


jlebar added a comment.

In http://reviews.llvm.org/D16250#328552, @tra wrote:

> Looks OK to me. 
>  Perhaps BoundArch w/o toolchain is sufficient for the key as toolchain would be derived from it.


Yeah, it might work, I wasn't convinced it was sound.  The bound TC is derived from the original TC plus the arch -- that is, the arch isn't the only piece of info that goes into it.  Dunno, once bitten twice shy.

Thank you for the reviews, and thanks again for your patience, Chris.


http://reviews.llvm.org/D16250





More information about the cfe-commits mailing list