[PATCH] D100776: [clang/Basic] Make TargetInfo.h not use DataLayout again

Nico Weber via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 19 13:36:35 PDT 2021


thakis added a comment.

In D100776#2699565 <https://reviews.llvm.org/D100776#2699565>, @rnk wrote:

> Of the three people who commented on D17183 <https://reviews.llvm.org/D17183>, you and I are on the only ones in favor of this approach. I think @echristo and @jyknight both preferred approach 2. I'd like to get at least an agreement to disagree from one of them before approving this.

That's 50% in people, and 15/24 voting by changes that landed in clang/ since start of year ;) Don't know why 2/2 is "only".

I'm also not dismissing approach 2: I implemented it as well, and it's ready-to-go as soon as someone wants to pursue the direction mentioned there. If someone wants to do that, I don't have a problem with it, but until then this here is simpler and more self-consistent since it preserves the original design of TargetInfo.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D100776/new/

https://reviews.llvm.org/D100776



More information about the cfe-commits mailing list