[PATCH] D58328: [X86] Fix tls variable lowering issue with large code model

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 17 16:12:23 PST 2019


craig.topper added a comment.

It looks like this review was not subscribed to llvm-commits when it was created. This prevents the general mailing list from being notified of its existence. Can you abandon this and submit a new review with llvm-commits subscribed from the start?

It's also helpful to start reviews with a tag like "[X86]" that indicates the area of the compiler you're changing. It helps potentially interested people search through the many emails on llvm-commits and find the things they might be interested in.


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

https://reviews.llvm.org/D58328





More information about the llvm-commits mailing list