[PATCH] D68063: Propeller: LLVM support for basic block sections

Sriraman Tallam via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 9 17:17:24 PDT 2020


tmsriram added a comment.

In D68063#1913544 <https://reviews.llvm.org/D68063#1913544>, @echristo wrote:

> Couple of inline comments. I'm trying pretty hard to be able to get rid of TargetOptions.h some day if possible. Any thoughts on ways to do this without?


Missed this comment.  I can dig more and maybe move it to TargetMachine.h potentially.  This has to be done from llc and LTO too so not sure how that would work.  Where were you planning on moving the existing fields of TargetOptions to?  Thanks.


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

https://reviews.llvm.org/D68063





More information about the llvm-commits mailing list